MinayeAliyeva / Foodie-App

https://foodie-app-xi.vercel.app
0 stars 0 forks source link

Review #30

Open Javahzade opened 2 months ago

Javahzade commented 2 months ago

Prod rejimində console-lar qalıb (Çoxlu sayda). Pagination məntiqi düzgün işləmir. Hər ekranda rerenderlər baş verir. Ana sahvəyə klik etdikdə tam reload baş verir. Favorites ekranında məhsulu unlike edəndə list ümumilikdə yenilənir. (Əlavə delay verməyə ehtiyac yox idi)

Lazımsız hook-dur. export const usehMap = (routes: MyRouterObject[]) => { return routes.map((route) => { return route; }); };

Bir çox komponentlərin qarşısında X yazısı var, başqa adlandırma seçmək olardı. Redux istifadəsi daha məsləhətli idi.

Total score: 82

MinayeAliyeva commented 2 months ago

Paginationu test meqsedli yazdigimi(qalsin amma yoxlamayin sadece logice baxarsiz dediyimi xatirlayiram) yoxlanmamasii bildirmisdim . Favorileri ozum server side olsun deye yazmisam cunki digerlerinde ele idi favorinide o mentiqe kecirdim. Component adinda ise ozumcun X componentleri custom component hesab etmisem basga ad ne tovsiye ederdiz?. Lazımsız hook-du==> Bunu elave olarag men login register private routing yazacam taskin davami olarag onuncun yazmisam.Redux isdifadesi meslehetlidir neyi nezerde tutursuz?