Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(lecture-5): Сервисы #383

Open
wants to merge 7 commits into
base: lecture-5
Choose a base branch
from

Conversation

gitSergeyhab
Copy link

Задачи

  • На странице товара добавил пользовательские комментарии
  • Сделал заглушки для неавторизованного пользователя
  • Сделал форму для добавления комментов авторизованными пользователями
  • Добавил стор на редакс для комментов

Дополнительно

  • Сделал сервис I18n
  • Переделал хук useTranslate
  • Добавил смену языка в апи
  • Добавил перезагрузку переводных данных

Объяснение "архитектурного решения" дополнительной задачи

  • так проще
09.10.2024.16.48.56.webm

- На странице товара добавить пользовательские комментарии
- Сделать заглушки для неавторизованного пользователя
- Сделать текстарею для добавления комментов авторизованными пользователями
- Добаить стор на редакс для комментов
- Слелал сервич I18n
- Переделал хук useTranslate
- Добавил смену языка в апи
- Добавил перезагрузку переводных данных
- убрать отступы в листе для вложенности больше 5
- добавить перенос для длинных слов
- сделать имя авторизованного автора коммента серым
- показывать форму туда, где окажется текст ответа
- убрать плейсхолдер из формы
- добавить автоскрол к форме
- сделать редирект к комментариям при переходе по ссылке "войдите" и входа
- применить listToTree для комментов
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant