Sprint 6/step 1 #8
Conversation
There was a problem hiding this comment.
Здравствуйте. Работа проделана огромная
- все 40 юнит-тестов проходят успешно
- сайт задеплоен и отлично работает
но есть некоторые недочеты:
- 4
cypress-теста не прошли. Скрин http://joxi.ru/J2bzbWltgkVKK2
Можно лучше
- лучше отображать эффект загрузки при ожидании ответа от сервера с номером заказа.
- Чтобы не дублировать типы экшенов и сами объекты с
type, принято выносить их в отдельные функции, называемыеaction creator(функцию, которая возвращает объект сtype). Это позволяет сделать понятное название для функции, которое будет говорить, что делает она, и избавит от повтора объекта сtypeв коде. Подробнее тут: https://rajdee.gitbooks.io/redux-in-russian/content/docs/basics/Actions.html#генераторы-экшенов-action-creators
Исправьте, пожалуйста, недочеты и работа будет принята. Пожалуйста, проверьте работоспособность проекта и наличие возможных ошибок в консоли браузера (кнопка F12) перед отправкой на ревью.
Напоминаю, что работа может быть принята только после исправления всех критических замечаний Нужно исправить.
Удачного рефакторинга кода.
There was a problem hiding this comment.
Поздравляю! Ваша работа принята.
Вы отлично потрудились.
- теперь все тесты проходят успешно
Удачи в новой профессии
Несколько советов для дальнейшего развития:
- сверстайте несколько небольших проектов для себя, для закрепления знаний, и чтобы можно добавить их в портфолио
- ходите на собеседования: опыт прохождения собеседований не менее важен знаний теории, и Вы поймете, какие знания требуются на рынке, и где Вы проседаете
- читайте habr и слушаете подкаксты: они очень хорошо расширяют кругозор и позволяют узнать, что происходит в отрасли:
https://soundcloud.com/everyonecan
https://soundcloud.com/podlodka
https://soundcloud.com/devschacht
https://soundcloud.com/begebot
| describe('burger ingredients drag and drop, create order', function () { | ||
| before(function () { | ||
| cy.visit('http://localhost:3000'); | ||
| cy.visit('http://localhost:3000/#'); |
There was a problem hiding this comment.
Вам надо было сделать testUrl в виде константы, чтобы 1 раз можно было изменить и везде изменилось бы
2 participants
No description provided.