Skip to content

Conversation

@dinara-n
Copy link
Collaborator

@dinara-n dinara-n commented Jan 8, 2023

  1. Task: https://github.com/rolling-scopes-school/tasks/tree/master/tasks/online-store-team
  2. Screenshot:
    screenshot-10-01-2023
  3. Deploy: https://buffik.github.io/Online-Store/#/
  4. Done 10.01.2023 / deadline 10.01.2023
  5. Score: 500 / 500

Cross-check: 300 / 300

Mentor's check: 200 / 200

  1. Репозиторий:
  • 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 баллов (процент покрытия каждой функции/метода не учитывается
    Комментарий: покрыто тестами 10 функций * 2 балла = 20 баллов.

Buffik and others added 30 commits December 26, 2022 23:55
feat: add template of cart, add rendering products from server, add +…
dinara-n and others added 22 commits January 11, 2023 13:03
…st to the component ProductCartButton.tsx, add test to utils functions (#25)
…ing from API, while changing count of products in cart (#27)
* fix: revert old logic with two states of cart to prevent plural fetching from API, while changing count of products in cart

* fix: lint errors at carousel imgs, lint errors in wrappers of modal window, delete unused components
dinara-n and others added 3 commits January 31, 2023 12:57
* fix: response types at API services

* feat: fix types at util function handle localstorage
@Buffik Buffik closed this Jan 31, 2023
@Buffik Buffik reopened this Jan 31, 2023
Copy link

@ConstantinPR0 ConstantinPR0 left a comment

Choose a reason for hiding this comment

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

В целом очень хороший проект, мне понравилось что присутствуют переиспользуемые компоненты и для первого проекта на React весьма грамотно организована структурированность компонентов

P.S. В проекте используется scss но не используется никаких сильных сторон препроцессора, будто бы он добавлен чисто для галочки, рекомендую изучить основные плюсы использования scss, если продолжать писать в таком духе, как сейчас, то обычный css выигрывает в производительности.

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