Code review aplikacji napisanej w Reakcie przez junior developera (część 7/10 - omówienie refactoru)

Moim gościem był Mati, junior developer, który w ramach nauki stworzył aplikację - listę zakupów. W nagraniu omawiamy refactor, nad którym spędziłem ostatnie dwa dni. Zapraszam do oglądania!

Linki:

00:00:00 - Start
00:01:10 - wstęp
00:02:50 - w następnych odcinkach…
00:03:38 - eslint i prettier
00:10:25 - ujednolicenie nazw plików
00:11:12 - aktualizacja paczek
00:18:29 - craco
00:21:40 - testy
00:33:31 - husky
00:40:57 - conventional commits
00:42:52 - naprawa zależności
00:46:38 - tłumaczenia
00:55:09 - wydzielenie “routes” do jednego pliku
00:56:26 - wydzielenie globalnych zmiennych
00:59:27 - zwiększenie czytelności kodu poprzez zastosowanie statycznych właściwości dla tagów html
01:03:20 - zastosowanie DRY do formularza
01:08:57 - powiadomienia - wydzielenie do osobnej biblioteki
01:10:54 - nieużywany kod
01:12:13 - formularze
01:24:14 - fix - zmiana języka
01:24:40 - nazewnictwo propsów w komponentach
01:26:38 - obsługa błędów w formularzach
01:28:11 - asynchroniczność w formularzach
01:30:24 - uproszczone api komponentu
01:34:22 - wydzielenie komponentów odpowiedzialnych za style
01:36:00 - pozostałe poprawki
01:40:28 - odpowiedzi na pytania Matiego
01:49:20 - flow vs typescript
01:51:31 - co jeszcze można poprawić?
01:52:45 - zakończenie

[xorbitxo]

Maga spoko film! Jeszcze nie jestem juniorem, ale z pewnością zastosuję część rzeczy z tego filmiku do własnych projektów.

[patrykbudnicki]

z tymi svg, to możesz zostawić fill'a, ale tylko w tagu svg. Te które są w <path>, to ich nie da się nadpisać przez css'y, te które są w <svg> bez problemu można :)

[TheKseon]

Bardzo fajny materiał. Z pewnością będę dalej śledzić dalsze materiały.

Miałbym takie pytanie niezwiązane z bieżącym odcinkiem. Co w sytuacji, kiedy mamy aplikacje fullstack, gdzie front jest React, a back-end w Express. Na obu używamy TypeScript i dzielą one część typów. Jak najlepiej współdzielić taką część kodu? Zastosować monorepo? A jeśli tak to, co z gitem? Git modules? Git trees?

[patrykbudnicki]

A propo jednego pliku routes – co uważasz o podejściu żeby zawierał on ścieżki tylko do modułów i każdy z tych modułów żeby miał już swój plik routes? Oczywiście wedle tej samej konwencji. Chodzi mi o to, że z jednej strony tworzymy aplikacje gdzie moduły są osobnymi bytami, które w dosyć prosty sposób moglibyśmy wyciągnąć z naszej aplikacji i wkleić je do innej, a z drugiej strony mamy jeden plik routes który wie wszystko o wszystkich modułach.