-
Notifications
You must be signed in to change notification settings - Fork 4
Refaktor komponentów
KatarzynaChmielecka edited this page Feb 9, 2022
·
1 revision
Przy refaktorowaniu komponentów do rozważenia:
- kto może zobaczyć dany komponent? czy niezalogowana osoba z linku też? czy jest on dostępny dla wszystkich ról? czy tylko np. obsługa? (jeśli informacje są niejawne - blokować do nich dostęp nieupoważnionym)
- generalnie gdzie nie trzeba się odwoływać do url, to z tego nie korzystamy (jeśli możemy wyciągnąć potrzebne informacje inną drogą)
- czy strona wyrenderuje się zgodnie z oczekiwaniami jak ktoś wejdzie z innej strony lub z linka
- wszędzie gdzie wyciągacie dane z mocków (czyli nasza atrapa pobierania danych z DB) —> użwyajcie useEffect!!! + zabezpieczcie sytuację co jak się fetch nie wykona a co jak fetch się wykona, ale dostaniemy pusty obiekt
- jeśli pobieracie jakieś dane, co jak np będą luki / nie będzie w danych np jakiegoś prop-value którego potrzebujecie
- raczej starać się jak najwięcej logiki robić w jak najwyższych/największych komponentach (najwięcej w pagebody, najmniej w atomkach - w takim zakresie jak to jest możliwe)
- jeśli macie jakieś pytania / niepewność / sugestie odnośnie np. path do Waszego komponentu, działania jakiegoś innego już stworzonego komponentu, zobaczycie że Waszą stronę można gdzieś jeszcze wykorzystać, trzeba rozwiązać globalnie problem z jakimś dynamicznym stanem itp, itd - od razu rzucajcie temat, bo dużo robiliśmy z myślą o późniejszym refaktorze, dużo można poprawić więc co wyłapiemy szybciej to będzie kosztowało ostatecznie mniej przeróbek później i w mniejszym zakresie
- Przy tworzeniu CSS / modyfikowaniu CSS uwzględniajcie od razu stylelint i theme
Jeżeli chcesz zaktualizować front na vercelu to pamiętaj przed pushnięciem swojego commita o uruchomieniu w głównym folderze komendy: npm run build
.
W folderze client/build
zbuduje się strona i stamtąd vercel sobie pobiera front.