mormolad / movies-explorer-frontend

https://movies-explorer-frontend-five-delta.vercel.app
1 stars 0 forks source link

1 review #25

Open mormolad opened 7 months ago

mormolad commented 7 months ago

При поиске текст запроса, найденные фильмы и состояние переключателя короткометражек сохраняются в хранилище. Если пользователь повторно переходит на страницу фильмов, то при монтировании компонента данные достаются из локального хранилища. Страница отображается в соответствии с загруженными из хранилища данными. Комментарий: В вашем задании указано: при каждом поиске на странице фильмов результаты поиска фильмов нужно сохранять в localstorage. В сохраняемые данные должны входить и строка поиска и статус чекбокса «Короткометражки», чтобы пользователю при повторном переходе на страницу фильмов, можно было вывести на экран информацию, полностью совпадающую с его последним поиском (с расстановкой актуального состояния лайков) перед уходом на другую страницу или перезагрузкой страницы. Эту информацию можно будет доставать из хранилища при монтировании и перерисовке компоненты фильмов. При логауте пользователя нужно эту информацию в хранилище и стейте (useState, если она там есть) уничтожать. Комментарий: Сейчас пользователь не видит точного результата своего последнего поиска на странице фильмов при повторном переходе на эту страницу с другой страницы, если при поиске был использован чекбокс Короткометражка. Положение чекбокса так-же не восстанавливается. Комментарий: Сохранять данные последнего поиска на странице «Сохранённые фильмы» для их показа при повторном входе на эту страницу не нужно. https://skr.sh/sL71AR8mTAd Нужно, чтобы при входе на страницу «Сохранённые фильмы» пользователь видел все свои сохранённые карточки без задания параметров поиска.


_Работа с фильтром настроена: Поиск фильмов регистронезависимый.

Если карточки уже были отображены на странице в блоке результатов, клик по чекбоксу «Короткометражки» приводит к повторной фильтрации результата. Комментарий: https://skr.sh/sNazwteLo9O Если поиск произвести с включенным чекбоксом, то в результате показаны все фильмы, без учета их длительности_


Карточки: Кнопка лайка имеет правильное состояние, в зависимости от того, добавлен ли фильм в сохранённые или нет. Комментарий: https://skr.sh/vNf9bEe3yoK После поиска лайки на фильмах отображаются некорректно


На странице «Сохранённые фильмы»: Отображается форма поиска. Она позволяет искать фильмы по уже полученным данным о сохранённых фильмах. Комментарий: Если выполнить поиск фильмов и затем в блоке результата удалить один фильм, то несмотря на наличие поискового запроса в строке поиска, в блоке результата будут отражены все сохраненные фильмы.


Удалённые или добавленные карточки фильмов появляются только после перезагрузки страницы. Комментарий: если фильм был ранее сохранен, то иконка лайка должна остаться активной, если этот фильм появился в поиске


Не выполняются лишние запросы к бэкенду, например: запрос всех фильмов с сервиса beatfilm-movies производится только при первом поиске, все сохранённые фильмы не запрашиваются с сервера при каждом лайке или дизлайке, данные пользователя запрашиваются один раз при запуске приложения. Комментарий: При каждом лайке происходит дополнительное обращение за получением данных с movies


Для внутренних ссылок в приложении используются компоненты из библиотеки react-router. Комментарий: на страницах с фильмами не отображается активная страница, используйте NavLink для задания активности


поля формы заблокированы во время отправки запросов и у пользователя нет возможности отправить новый запрос до завершения предыдущего. Комментарий: на время выполнения запроса считается хорошей практикой блокировать поля ввода и кнопку отправки формы, что бы пользователь не мог выполнить новые запросы до завершения предыдущего