CR Frontend #3

[Arsenicro]

• [x] https://github.com/ItachiEU/BlogSite/blob/33dc9d027c57cb33f4a4297a6c9dafc81df33ac8/blog/frontend/src/App.tsx#L15 Po co ten tag?
• [ ] https://github.com/ItachiEU/BlogSite/blob/32952d2e9b806d61dd8985fe0a4212597e89fe3b/blog/frontend/src/components/Contact/Contact.tsx#L32 required bez rules to wallidacja tylko wyglądu, niefunkcjonalna
• [ ] https://github.com/ItachiEU/BlogSite/blob/076c3fc25eedc6a901058dee60eb850c84162fe4/blog/frontend/src/components/NewArticle/NewArticle.tsx#L18 Co jeśli ten fetch się nie uda? Stworzycie nowy artykuł bez tagu? Co jeśli ten request "zgubi się" w sieci, i tworzenie artykułu zadzieje się wcześniej? Zróbcie tu chociaż awaita
• [ ] https://github.com/ItachiEU/BlogSite/blob/076c3fc25eedc6a901058dee60eb850c84162fe4/blog/frontend/src/components/NewArticle/NewArticle.tsx#L18 Czemu właściwie mieszacie fetcha z axiosem?
• [ ] https://github.com/ItachiEU/BlogSite/blob/076c3fc25eedc6a901058dee60eb850c84162fe4/blog/frontend/src/components/NewArticle/NewArticle.tsx#L38 To śmieszne, ale useCallback oczekuje inlinowej funkcji jako argumentu. Przez to wali wam warrningiem. Jedyny sensowny sposób jaki widzę żeby to ominąć, to użyć useMemo zamiast useCallback i podać tam funkcję (tzn useMemo(() => debounce(...)))
• [x] https://github.com/ItachiEU/BlogSite/blob/076c3fc25eedc6a901058dee60eb850c84162fe4/blog/frontend/src/components/NewArticle/NewArticle.tsx#L35 A co ja mówiłem o ustatwianiu stanu jak zależy on od poprzedniego? Użyjcie funkcji ustawiającej stan
• [x] https://github.com/ItachiEU/BlogSite/blob/076c3fc25eedc6a901058dee60eb850c84162fe4/blog/frontend/src/components/NewArticle/NewArticle.tsx#L81 Tak nie robimy. Jak handleFileInputChange dacie normalny typ, a nie ten wasz, tzn React.ChangeEvent<HTMLInputElement>, to po małej zmianie (e.target.files?.[0]) wszystko będzie śmigać bez żadnych haków
• [x] https://github.com/ItachiEU/BlogSite/blob/33dc9d027c57cb33f4a4297a6c9dafc81df33ac8/blog/frontend/src/components/BlogList/BlogList.tsx#L28 brak zależności
• [x] https://github.com/ItachiEU/BlogSite/blob/33dc9d027c57cb33f4a4297a6c9dafc81df33ac8/blog/frontend/src/components/BlogList/BlogList.tsx#L42 Cały czas tak nie robimy, naprawcie typy
• [ ] https://github.com/ItachiEU/BlogSite/blob/33dc9d027c57cb33f4a4297a6c9dafc81df33ac8/blog/frontend/src/components/BlogList/BlogList.tsx#L27 Czemu tak? Jest mnóstwo jednostek css'owych które robią podobne rzeczy, typu vw albo % albo coś innego https://www.w3schools.com/cssref/css_units.asp

[Arsenicro]

Jeszcze rzeczy które znalazłem przed chwilą:

• [ ] https://github.com/ItachiEU/BlogSite/blob/076c3fc25eedc6a901058dee60eb850c84162fe4/blog/frontend/package.json#L3 Proxy nie będzie wam działać na buildzie produkcyjnym, musicie CORSa rozwiązać po stronie backendu
• [ ] https://github.com/ItachiEU/BlogSite/blob/33dc9d027c57cb33f4a4297a6c9dafc81df33ac8/blog/frontend/src/components/BlogList/BlogList.tsx#L39 To nie zadziała, bo za każdym rerenderowaniem funkcja v4 wywoła się ponownie, więc wygeneruje się nowy klucz, więc klucze za każdym razem się zmienią. Lepiej już by było dać zwyczajnie indeksy, ale to też jest słabe. Każdy post powinien mieć ID
• [ ] https://github.com/ItachiEU/BlogSite/blob/224809b3b4b9fe394bd8855f4a1ee854c396db71/blog/frontend/src/atoms/atoms.ts#L4 Co tu robią te domyślne wartości jak już jest backend?