Size: a a a

2021 June 02

p

persona x grata in JS Liberty
Задание проверили, но к сожалению, не готовы перейти к дальнейшим шагам, так как к заданию есть замечания. Ревью:

Общее впечатление
Приложение работает некорректно. На данном этапе мы останавливаемся, т.к. тестовое задание выполнено с критичными для нас минусами.

Плюсы

Есть пример конфига
Хорошая и понятная организация кода
Минусы

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

Хотели бы поблагодарить Вас за время уделенное нашей компании.
источник

p

persona x grata in JS Liberty
До какой-то хуйни подоебывались)
источник

x

xDaemonic in JS Liberty
А это на какую позицию с таким тестовым?
источник

p

persona x grata in JS Liberty
Мидл
источник

x

xDaemonic in JS Liberty
Рили? Это задание на мило реактора разраба? А оклад какой?
источник

p

persona x grata in JS Liberty
от 150
источник

x

xDaemonic in JS Liberty
Ахренеть
источник

p

persona x grata in JS Liberty
Ну не сложное задание. Только ТЗ составлено максимально хуево
источник

x

xDaemonic in JS Liberty
И я про то же, что не сложно... Пора мне работу менять 😂
источник

p

persona x grata in JS Liberty
Переслано от Th0r
мне такое прислали
источник

p

persona x grata in JS Liberty
Похожее на сеньера)
источник

ia

ivan anatolyevich 🏋... in JS Liberty
спиздили твой проект и продали ) а тебе типа тест бесплатно 😭
источник

p

persona x grata in JS Liberty
Пусть подавятся говнокодом))
источник

A

Arseny in JS Liberty
А шо такое radiobatton?  Новый сорт хлеба шоле?
источник

T

Th0r in JS Liberty
Загадочная история Бенджамина Баттона
источник

ia

ivan anatolyevich 🏋... in JS Liberty
источник

A

Arseny in JS Liberty
ааа грузинский связист
источник

A

Arseny in JS Liberty
До феншуя в основном, вот без него и не к чему было бы)
источник

T

Th0r in JS Liberty
почему довыебывались-то?
критичные вещи то есть,
а так - тебе дали отличный фидбек, даже по мелким недостаткам рассказали
источник

YP

Yarik Pesochenski in JS Liberty
То что смутило меня:
1. А было оговорено что надо было написать тесты? Мне в моем тестовом прямо указали и я все сделал
2. Не понял доебки насчет двух lock файлов и консоль логов
3. Не понял насчет распределения зависимостей, хотя момент конечно и спорный. но с другой стороны какая нахуй разница
источник