search

TecHeReTiC3141 / SHRI_React_HW

0 stars 0 forks source link

Ревью #2

Open nchertolin opened 2 months ago

nchertolin commented 2 months ago

Build

Билды не собирается в обоих ветках

npm run build
nchertolin commented 2 months ago

FSD

  1. Странно, что конфиг store находиться в entities/film. Я его обычно выношу на уровень app
  2. API, с которым взаимодействует слайс, лучше делать с помощью injectEndpoints в этом же слайсе.(мое мнение) Например: features/star-rating/api/index.ts. В нем добавляешь эндпоинт на оценку фильма.
nchertolin commented 2 months ago

ENV

Лучше вынести адрес сервера в переменную окружения.

VITE_API=http://localhost:3030/api/v1
nchertolin commented 2 months ago

Summary

✅ 100/100

Все пункты выполнены успешно! Посмотрел у тебя много интересных фич.

Сам использовал MUI, надеюсь не задушат))