p
Задание проверили, но к сожалению, не готовы перейти к дальнейшим шагам, так как к заданию есть замечания. Ревью:
Общее впечатление
Приложение работает некорректно. На данном этапе мы останавливаемся, т.к. тестовое задание выполнено с критичными для нас минусами.
Плюсы
Есть пример конфига
Хорошая и понятная организация кода
Минусы
Нет ссылки на собранный проект (надо выкачивать проект и собирать вручную)
Нет тестов
Неправильное разделение зависимостей на dependencies и devDependencies (в одном месте собраны все npm пакеты, которые нужны для самого приложения и которые нужны только для разработки)
Не зафиксированы версии npm пакетов (большая вероятность использования разных версий npm пакетов в командной работе и при использовании CI/CD, что может привести, например, к трудности отладки)
В проекте есть package-lock.json и yarn.lock. Как минимум это странно и самое главное объявленные в них зависимости имеют разные версии
В консоли браузера есть ошибки приложения
В коде присутствуют console.log`и
Нет обработки ошибок пользователя
Не безопасное использование JSON.parse из-за этого приложение падает
Нет перехода на вкладку Result после нажатия на Apply
Не правильно работает контрол Radiobatton (например, нужно задать выбор цвета: жёлтый или чёрный, в текущей версии приложения можно выбрать оба цвета)"
Хотели бы поблагодарить Вас за время уделенное нашей компании.