BKonstantine/task_management_system

Ревью итогового задания

Opened this issue · 0 comments

  1. Необходимо запрашивать по 10 элементов по умолчанию. Для того чтобы отобразить пользователей например в проектах и задачах, после получения массива задач собираешь оттуда _id пользователей и запрашиваешь одним массивом на post users/search передав в фильтры массив нужных _id. Возможно это сделано для выпадашки пользователей еще,так как там не предусмотрена пагинация, но в таком случае лучше было запросить только первые 10, и попросить добавить пагинацию в макеты

    limit: data.total * 10,

  2. Не стоит в строке запроса хранить, не удается перейти ко всем задачам из навигации, хранить в сторе, чистить если перешли в задачи по навигации, а не через проект
    image

  3. Такие проверки в computed

    v-if="totalPage > 1 && projectsLength > 0"

  4. Данные для сортировок, фильтрации, список статусов все, что может понадобится в разных компонентах выносить в константы

  5. С таким названием могут возникнуть ошибки, ты отправляешь событие, а не навешиваешь клик на элемент, лучше clear

По функционалу в целом:

Авторизация:

  • Кнопка "Войти" должна быть не доступна для нажатия, если поля не заполнены
  • нет возможности зайти под другим пользователем, так как всегда подставляются зашитые данные
  • данные для входа вшиты, нет возможности залогинится под другим пользователем
  • не хватает глобального хука для проверки маршрутов, после выхода из системы и переход в ручную на страницу проектов происходит логин по зашитым данным и пускает в систему

Профиль:

  • не хватает функционала

Пользователи:

  • нет возможности удалить или отредактировать пользователя
  • нет возможности добавить пользователя

Задачи:

  • нет возможности удалить/редактировать задачу
  • не хватает функционала в карточке задачи

Проекты:

  • нет возможности отредактировать/удалить проект

Общие:

  • строка поиска присутствует, но не работает по иконке Лупы или по Enter. Также не срабатывает отчищение поиска по иконки крестика
  • не выделяется цветом совпадения при поиске
  • в пагинации нет возможности ввести страницу и перейти к ней
  • сортировка не срабатывает, если было выбрано значение из выпадающего списка, срабатывает только по стрелке общей сортировки
  • фильтры не реализованы