Путаница в рейтинге группы #76

Closed
opened 2026-09-14 23:31:46 +03:00 by NotBigGhost · 2 comments
Owner

Рейтинг группы не сходится с настоящим составом группы. Если из группы удалить участника, он останется в рейтинге без каких либо изменений, что вызывает путаницу. В рейтинге группы нужно отображать только тех игроков, которые являются её участниками в данный момент.

Рейтинг группы не сходится с настоящим составом группы. Если из группы удалить участника, он останется в рейтинге без каких либо изменений, что вызывает путаницу. В рейтинге группы нужно отображать только тех игроков, которые являются её участниками в данный момент.
NotBigGhost added the
Priority
High
2
Kind/Enhancement
labels 2026-09-14 23:31:46 +03:00
Agent self-assigned this 2026-09-14 23:40:40 +03:00
Agent added the
Reviewed
Confirmed
1
label 2026-09-14 23:40:42 +03:00
Collaborator

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

  1. Групповая цепочка рейтинга по-прежнему проигрывает все партии группы: партии с ушедшим игроком влияют на рейтинг оставшихся. Фильтруется только вывод.
  2. stats_service.leaderboard получает набор текущих участников. Игроки вне состава отбрасываются до сортировки, поэтому места нумеруются 1..n без дыр. Фильтр использует только group_stats; общий топ и главная не меняются.
  3. total_matches, дата последней партии и статистика фракций группы остаются как есть: это статистика партий, а не состава.
  4. На фронте после удаления участника обновляется и статистика группы, не только список участников.
  5. Тесты: удалённый пропадает из списка группы, но остаётся в общем топе; рейтинг оставшегося учитывает партию с ушедшим; при возврате в группу игрок снова в списке; ранги идут без пропусков.

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

  • В статистике группы нет игроков вне текущего состава.
  • Рейтинг оставшихся учитывает партии с ушедшими; общий топ без изменений.
  • pytest и npm run build зелёные.

Ветка: issue-76-group-rating-members от dev (в задаче не указана целевая ветка — взята dev, как для всех фич)

## План выполнения 1. Групповая цепочка рейтинга по-прежнему проигрывает **все** партии группы: партии с ушедшим игроком влияют на рейтинг оставшихся. Фильтруется только вывод. 2. `stats_service.leaderboard` получает набор текущих участников. Игроки вне состава отбрасываются до сортировки, поэтому места нумеруются 1..n без дыр. Фильтр использует только `group_stats`; общий топ и главная не меняются. 3. `total_matches`, дата последней партии и статистика фракций группы остаются как есть: это статистика партий, а не состава. 4. На фронте после удаления участника обновляется и статистика группы, не только список участников. 5. Тесты: удалённый пропадает из списка группы, но остаётся в общем топе; рейтинг оставшегося учитывает партию с ушедшим; при возврате в группу игрок снова в списке; ранги идут без пропусков. **Критерии готовности** - В статистике группы нет игроков вне текущего состава. - Рейтинг оставшихся учитывает партии с ушедшими; общий топ без изменений. - `pytest` и `npm run build` зелёные. **Ветка:** `issue-76-group-rating-members` от `dev` (в задаче не указана целевая ветка — взята `dev`, как для всех фич)
Agent added the
Status
In Review
label 2026-09-14 23:47:40 +03:00
Collaborator

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

Итог: в «Списке игроков» группы — только текущий состав. Партии с ушедшими игроками остаются в групповой цепочке, поэтому рейтинг оставшихся не меняется. Общий топ без изменений.
Проверки: pytest — 191 passed (2 новых теста); npm run build — зелёный.
Статус: Status/In Review
Осталось за вами: ревью и мёрж PR — задача закроется автоматически.

Работа выполнена, открыт PR: https://gitea.arseniev.info/NotBigGhost/ForbiddenStarsApp/pulls/78 **Итог:** в «Списке игроков» группы — только текущий состав. Партии с ушедшими игроками остаются в групповой цепочке, поэтому рейтинг оставшихся не меняется. Общий топ без изменений. **Проверки:** `pytest` — 191 passed (2 новых теста); `npm run build` — зелёный. **Статус:** `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#76