Ревью тостов #41

Closed
opened 2026-09-09 20:22:06 +03:00 by NotBigGhost · 2 comments
Owner

Просмотреть существующие в приложении тосты, убрать лишние.

Просмотреть существующие в приложении тосты, убрать лишние.
NotBigGhost added the Kind/Enhancement label 2026-09-09 20:22:06 +03:00
NotBigGhost added the Kind/Code Review label 2026-09-13 13:23:36 +03:00
Agent self-assigned this 2026-09-13 15:37:53 +03:00
Agent added the
Reviewed
Confirmed
1
label 2026-09-13 15:37:54 +03:00
Agent added the
Status
Need More Info
2
label 2026-09-13 15:42:29 +03:00
Agent removed the
Status
Need More Info
2
label 2026-09-13 15:43:53 +03:00
Collaborator

План выполнения

Правило: тост лишний, если результат действия и так виден на экране — значение изменилось, элемент появился или исчез, форма закрылась, произошёл переход на страницу результата.

Остаются:

  • ошибки;
  • предупреждения (лимит фото, повторы фракций, конфликт правок, «вы не в группе»);
  • действия без видимого следа («Приглашение отправлено», «Дополнения сохранены», «Вы вступили в группу»);
  • сообщение перед уходом на другую страницу (удаление партии);
  • сообщения с инструкцией («сообщите пароль игроку»);
  • сохранение текстовых полей, где заметна только погасшая кнопка.
  1. Убрать 19 тостов:
    • любимая фракция, аватар, ник, привязка Telegram;
    • «Группа создана», «Приглашение отклонено»;
    • «Партия начата / завершена / обновлена»;
    • в админке: ник, вкл./откл., ачивки, удаление записей и аккаунта.
  2. Дополнение владельца: оставшиеся тосты показывать как всплывашки уведомлений приложения — карточкой сверху экрана, в одной стопке с входящими уведомлениями.
    • Ошибки выделяются красной полосой и держатся дольше.
    • Одинаковые сообщения не копятся.
    • Клик по карточке закрывает её.
  3. NotificationToaster больше не рисует свою стопку, а отдаёт уведомления в общую. Сервер не шлёт автору уведомлений о его действиях, поэтому дублей с сообщениями нет.

Критерии готовности

  • Лишние тосты убраны, остальные на месте.
  • Нижнего тоста в приложении больше нет: сообщения, ошибки и уведомления идут сверху одной стопкой.
  • npm run build зелёный.

Ветка: issue-41-toast-review от dev

## План выполнения **Правило:** тост лишний, если результат действия и так виден на экране — значение изменилось, элемент появился или исчез, форма закрылась, произошёл переход на страницу результата. **Остаются:** - ошибки; - предупреждения (лимит фото, повторы фракций, конфликт правок, «вы не в группе»); - действия без видимого следа («Приглашение отправлено», «Дополнения сохранены», «Вы вступили в группу»); - сообщение перед уходом на другую страницу (удаление партии); - сообщения с инструкцией («сообщите пароль игроку»); - сохранение текстовых полей, где заметна только погасшая кнопка. 1. **Убрать 19 тостов:** - любимая фракция, аватар, ник, привязка Telegram; - «Группа создана», «Приглашение отклонено»; - «Партия начата / завершена / обновлена»; - в админке: ник, вкл./откл., ачивки, удаление записей и аккаунта. 2. **Дополнение владельца:** оставшиеся тосты показывать как всплывашки уведомлений приложения — карточкой сверху экрана, в одной стопке с входящими уведомлениями. - Ошибки выделяются красной полосой и держатся дольше. - Одинаковые сообщения не копятся. - Клик по карточке закрывает её. 3. **`NotificationToaster`** больше не рисует свою стопку, а отдаёт уведомления в общую. Сервер не шлёт автору уведомлений о его действиях, поэтому дублей с сообщениями нет. **Критерии готовности** - Лишние тосты убраны, остальные на месте. - Нижнего тоста в приложении больше нет: сообщения, ошибки и уведомления идут сверху одной стопкой. - `npm run build` зелёный. **Ветка:** `issue-41-toast-review` от `dev`
Agent added the
Status
In Review
label 2026-09-13 15:46:07 +03:00
Collaborator

Работа выполнена, открыт PR: #55

Итог:

  • Убрано 19 тостов, которые повторяли видимый результат действия.
  • Оставшиеся (ошибки, предупреждения, действия без видимого следа) всплывают сверху, как уведомления приложения, в одной стопке с ними. У ошибок красная полоса.

Проверки:

  • npm run build зелёный.
  • toast.show было 63, стало 12, плюс 32 toast.error.
  • Вживую не смотрел: расширение Chrome не подключено.

Статус: Status/In Review
Осталось за вами: посмотреть всплывашки на телефоне, ревью и мёрж PR — задача закроется автоматически.

Работа выполнена, открыт PR: https://gitea.arseniev.info/NotBigGhost/ForbiddenStarsApp/pulls/55 **Итог:** - Убрано 19 тостов, которые повторяли видимый результат действия. - Оставшиеся (ошибки, предупреждения, действия без видимого следа) всплывают сверху, как уведомления приложения, в одной стопке с ними. У ошибок красная полоса. **Проверки:** - `npm run build` зелёный. - `toast.show` было 63, стало 12, плюс 32 `toast.error`. - Вживую не смотрел: расширение Chrome не подключено. **Статус:** `Status/In Review` **Осталось за вами:** посмотреть всплывашки на телефоне, ревью и мёрж PR — задача закроется автоматически.
Sign in to join this conversation.
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: NotBigGhost/ForbiddenStarsApp#41