Skip to content

Conversation

@miptleha
Copy link
Owner

No description provided.

Copy link

@vadmas vadmas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Хорошая работа.
Приложение приобретает готовый вид, видно как вы прогрессируете по мере прохождения курса.
В положительную сторону хочется отметить:

  • Работа с модальными окнами реализована через порталы - отлично!
  • Запрос на получение списка ингредиентов отрабатывает корректно
  • В проекте есть все необходимые компоненты

Но есть несколько замечаний:

Замечания которые необходимо будет исправить:

  • Целесообразно выносить логику по работе с внешним API (запросами на сервер) в отдельные сущности (утилиты).
    Смотри комментарий в App.tsx

Так же стоит учесть комментарии, оставленные в коде.

Пожалуйста, учтите выше описанные замечания и те, что оставлены коде вашей работы (особенно с пометкой "нужно исправить").
При повторной проверке, если все замечания будут исправлены, работа будет принята.

Удачного рефакторинга кода!

@miptleha
Copy link
Owner Author

поправил app.jsx

@vadmas
Copy link

vadmas commented Jan 15, 2023

Отлично, все критичные комментарии исправлены. Работу принимаю. Удачи в дальнейшем обучении!

@miptleha miptleha merged commit 629e803 into main Jan 15, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants