Skip to content

Conversation

@jerubrin
Copy link
Owner

@jerubrin jerubrin commented Jan 7, 2023

  1. Ссылка на задание.

  2. Скриншот результата выполнения задания:

online-store

  1. Ссылка на задеплоенную версию вашего приложения или сайта.

  2. Дата сдачи / дата дедлайна.
    09.01.2023/10.01.2023

  3. Ваша самопроверка с предварительной оценкой.

Функционал приложения (300/300)

  • Страница товаров с фильтрами (+120)
  • Страница корзины выбранных товаров (+60)
  • Модальное окно оформления(покупки) выбранных товаров (+50)
  • Страница с описанием товара (+40)
  • Header, содержащий кол-во добавленных в корзину товаров и общую сумму покупок (+20)
  • Страница 404 (+10)

Проверка задания ментором (200/200)

  1. Репозиторий +20
  • pull request выполнен в соответствии с требованиями +10
  • ведётся история коммитов, названия коммитов даются согласно гайдлайну +10
  1. Качество кода +160
  • приложение написано на TypeScript.
    • используется Everyday Types +10
    • используются Generics +10
    • использование Object Types +10
    • использование Classes +10
    • использование Function +10
    • нигде не используется тип Any +10
    • ESLinter настроен на TypeScript (используется плагин typescript-eslint/recommended) и отсутствуют ошибки +10
    • В конфигурационном файле TypeScript стоят флаги "noImplicitAny": true и "strict": true + 20
    • webpack настроен и работает с TypeScript +10
  • код разбит на модули +20
  • карточки товаров добавляются динамически средствами JavaScript +10
  • у ментора нет замечаний к качеству кода, либо все замечания ментора исправлены +30
  1. Тесты в приложении +20
  • реализованы юнит-тесты, использующие различные методы jest – 2 балла за каждую покрытую функию/метод, но не более 20 баллов (процент покрытия каждой функции/метода не учитывается)

jerubrin and others added 30 commits December 13, 2022 21:25
Add base router and render into components
feat:add loading and render,sort and type of rendering cards
feat:add loading and render,sort and type of rendering cards
Add getter for products with query params filter
feat:add filters, searching, sorting
feat:add modal window and all validates
jerubrin and others added 23 commits January 8, 2023 19:28
feat: add wrong symbols checker into cvv
fix: wrong cart and date input
fix: wrong phone, card number and date.
Hot-fix: products in query params
Copy link

@akimserik akimserik left a comment

Choose a reason for hiding this comment

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

Overall - very good, well done!

Some suggestions:

  • Add validation error descriptions under form fields
  • If possible, reduce code lines by 200-250, exporting some logic to helper/utils

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.

4 participants