Ревью итогового задания
Opened this issue · 0 comments
-
Необходимо запрашивать по 10 элементов по умолчанию. Для того чтобы отобразить пользователей например в проектах и задачах, после получения массива задач собираешь оттуда _id пользователей и запрашиваешь одним массивом на post users/search передав в фильтры массив нужных _id. Возможно это сделано для выпадашки пользователей еще,так как там не предусмотрена пагинация, но в таком случае лучше было запросить только первые 10, и попросить добавить пагинацию в макеты
-
Не стоит в строке запроса хранить, не удается перейти ко всем задачам из навигации, хранить в сторе, чистить если перешли в задачи по навигации, а не через проект
-
Такие проверки в computed
-
Данные для сортировок, фильтрации, список статусов все, что может понадобится в разных компонентах выносить в константы
-
С таким названием могут возникнуть ошибки, ты отправляешь событие, а не навешиваешь клик на элемент, лучше clear
По функционалу в целом:
Авторизация:
- Кнопка "Войти" должна быть не доступна для нажатия, если поля не заполнены
- нет возможности зайти под другим пользователем, так как всегда подставляются зашитые данные
- данные для входа вшиты, нет возможности залогинится под другим пользователем
- не хватает глобального хука для проверки маршрутов, после выхода из системы и переход в ручную на страницу проектов происходит логин по зашитым данным и пускает в систему
Профиль:
- не хватает функционала
Пользователи:
- нет возможности удалить или отредактировать пользователя
- нет возможности добавить пользователя
Задачи:
- нет возможности удалить/редактировать задачу
- не хватает функционала в карточке задачи
Проекты:
- нет возможности отредактировать/удалить проект
Общие:
- строка поиска присутствует, но не работает по иконке Лупы или по Enter. Также не срабатывает отчищение поиска по иконки крестика
- не выделяется цветом совпадения при поиске
- в пагинации нет возможности ввести страницу и перейти к ней
- сортировка не срабатывает, если было выбрано значение из выпадающего списка, срабатывает только по стрелке общей сортировки
- фильтры не реализованы