rolling-scopes-school / support

15 stars 3 forks source link

Cross-Check 'eCommerce Application Sprint#2. Cross-check' - 'KaterinaMysl' #920

Closed katemihalovskaya closed 11 months ago

katemihalovskaya commented 11 months ago
  1. A link to your deployed project (final version)

    A link to your deployed project (sprint#2)

  2. A link to the project repository on Github (final)

    A link to the sprint#2 Pull Request on Github.

  3. A link to the task

  4. A Link to the checklist for evaluating the task (if it exists)

  5. A screenshot of cross-checking marks Due to different time zones we missed the deadline and did not submit the task in time.

  6. A final score after self-assessment, with comments Score: 315 / 315 Login Page Implementation (130/130) Registration Page Implementation (120/120) Main Page Enhancements (10/10) Routing Implementation (30/30) Evaluation Criteria for Header (25/25)

  7. A cross-check score of your Score

haslie22 commented 11 months ago

Login Page 130/130


Input Validation 40/40

Небольшой момент по форме: если вы разрешаете автозаполнение формы сохраненными данными, было бы здорово предусмотреть этот момент, чтобы плейсхолдеры своевременно уезжали вверх и становились лейблами, иначе выходит вот так: https://i2.paste.pics/7233ef1d998608812252044cc089012b.png

Аналогично в форме регистрации

Integration with Authentication Service 45/45

Есть небольшое замечание: если я неверно ввожу один из кредов, оба инпута подсвечены красным с подписью "Incorrect email or password". Я понимаю, что допустила опечатку в пароле, изменяю пароль, красное сообщение под паролем пропадает, но оно остается в логине, а он-то у меня верный, и форму при этом я сабмитнуть не могу. Более того, по логике валидации (наличие символов в верхнем/нижнем регистре, цифр, корректный формат емейла и т.п.) в нем тоже все ок. В итоге я вынуждена идти в логин, убирать там один символ и добавлять его же, чтобы триггернуть валидацию поля. Это не очень удобно. Как альтернативу на будущее предлагаю сделать общую красную подпись для обоих полей, и убирать ее по инпуту в одно из двух полей, оставляя лишь собственную валидацию полей, так юзеру будет удобнее.


Redirection 30/30

Handle Authentication Token 10/10

Navigation to Registration Page 5/5

Registration Page 120/120


Input Validation 45/45

Integration with Authentication Service 25/25

State Management, Automatic Login, and Redirection 15/15

Integration with commercetools for User Profiles and Addresses 30/30

Navigation to Login Page 5/5

Main Page Enhancements 65/65


Centralized Navigation 10/10

Routing Implementation 27/30

Evaluation Criteria for Header 25/25

Penalties


Total: 315 / 315

haslie22 commented 11 months ago

@valerydluski @helenakrasnova done, check please

KaterinaMysl commented 11 months ago

@haslie22 Thank you for review and good advices!

kazymirT commented 11 months ago

@haslie22 Thank you for reviewing our missed sprint. Thank you for the detailed feedback and recommendations.