Size: a a a

Flood - русскоговорящее сообщество

2020 December 18

z

z̛e͏́͠r͜c҉ in Flood - русскоговорящее сообщество
За год мидлом стал посмотрите на него
источник

Nu

Null undefined in Flood - русскоговорящее сообщество
Tim
Конечно
Почему ещё не развернул тогда? Вам что отдельное приглашение. Ясно, мы вам перезвоним
источник

z

z̛e͏́͠r͜c҉ in Flood - русскоговорящее сообщество
Null undefined
Почему ещё не развернул тогда? Вам что отдельное приглашение. Ясно, мы вам перезвоним
В этом случае прямо говорят вы нам не подходите
источник

T

Tim in Flood - русскоговорящее сообщество
Кошкву развернул
источник

T

Tim in Flood - русскоговорящее сообщество
Дерево ты следующее 🌲
источник

Nu

Null undefined in Flood - русскоговорящее сообщество
z̛e͏́͠r͜c҉
В этом случае прямо говорят вы нам не подходите
Тима петушок
источник

z

z̛e͏́͠r͜c҉ in Flood - русскоговорящее сообщество
@TimIrwing а ты только фронт пишешь ?
источник

T

Tim in Flood - русскоговорящее сообщество
Ну да
источник

T

Tim in Flood - русскоговорящее сообщество
Пока что чисто вьюшная история
источник

z

z̛e͏́͠r͜c҉ in Flood - русскоговорящее сообщество
Бэк потрогай
источник

z

z̛e͏́͠r͜c҉ in Flood - русскоговорящее сообщество
А то ты получился какой то css мидл
источник

L

Lookarious in Flood - русскоговорящее сообщество
·   Не стали использовать vuex-simple из пресета

·   В колбэках неиспользуемые параметры event с немного странной типизацией (не работали с TS, это не критично)

·   Лишние декораторы @Emit() в App.tsx

·   Логика обработки событий в рутовом компоненте

·   Стили для календаря высчитываются в рутовом компоненте App.tsx и прокидываются через пропсы на несколько уровней ниже

·   Для большинства свойств нет типов

·   Нет ключей

·   Очень много инлайн стилей в шаблонах

·   Местами используются let вместо const , хотя переменная не перезаписывается
источник

L

Lookarious in Flood - русскоговорящее сообщество
Фидбек от озон
источник

L

Lookarious in Flood - русскоговорящее сообщество
Я наверное согласен с некоторыми вещами особенно с типизацией но то что я написал логику в App.tsx они даже не поняли что мой календарь работает и без логики, что это тупой компонент куда можно доделать свой функционал и это сделано для переиспользуемости
источник

T

Tim in Flood - русскоговорящее сообщество
ну видимо они наоборот хотели абстрактный переиспользуемый компонент, а не обложку, которой каждый раз надо снаружи что-то дописывать)
источник

T

Tim in Flood - русскоговорящее сообщество
вот мое, можешь посмотреть
https://github.com/TimIrwing/ozon-homework
источник

L

Lookarious in Flood - русскоговорящее сообщество
Tim
ну видимо они наоборот хотели абстрактный переиспользуемый компонент, а не обложку, которой каждый раз надо снаружи что-то дописывать)
Мдаа ну и ладно, я уже нашёл работу, ЗП конеч в начале не будет такой но прийдётса потерпеть несколько месяцев
источник

T

Tim in Flood - русскоговорящее сообщество
Calendar
👍 Сделано сразу два завания, хотя можно было сделать одно)
👍 Вся логика работы с todo в сторе

💬 Локализацию месяцев и дней недели можно не хардкордить, а сделать через Intl.DateTimeFormat
💬 БЭМ лишний с CSS Modules
💬 При расчёте дней, лучше формировать простой массив, а потом выводить его, а не добавлять сразу элементы

🤷‍♀️ Всё написано в одном файле - читается очень сложно
🤷‍♀️ Расчёт дней месяца можно было вынести в отдельные функции, а не нагромождать компонент

Calculator
👍 Не используется eval
👍 enum для экшенов
👍 Отличный грид - расположение не зависит от разметки

💬 Стор читается сложновато, можно было бы разбить его на более понятные функции

🤷‍♀️ Всё написано в одном файле - читается очень сложно
🤷‍♀️ Стор используется напрямую в шалонах
источник

L

Lookarious in Flood - русскоговорящее сообщество
Пасиб гляну
источник

T

Tim in Flood - русскоговорящее сообщество
вот такой фидбек был
источник