Skip to content

Feat team style#10

Closed
EnderDissa wants to merge 2 commits into
devfrom
feat-team-style
Closed

Feat team style#10
EnderDissa wants to merge 2 commits into
devfrom
feat-team-style

Conversation

@EnderDissa
Copy link
Copy Markdown
Member

No description provided.

EnderDissa and others added 2 commits May 8, 2026 02:30
material/color/name живут в config.yml: teamselect.teams.<n>, единый источник
правды для GUI выбора, лобби-скорборда и режимов. Добавлен публичный API
TeamStyle/TeamStyleProvider — моды читают стили оттуда вместо хардкода.

v4-миграция: переносит имена из messages.yml: ui.teamselect.teams.<n>.name
в config.yml: teamselect.teams.<n>.name (если в config ещё нет), затем
вырезает legacy-секцию из messages.yml. Existing моды визуально не меняются.

Voicechat-интеграция (TeamGroupManager) использует новый API для имени команды
вместо собственного парсера messages.yml.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 621a635cd4

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment on lines +76 to +78
val candidates = mutableSetOf<UUID>()
candidates += instance.getActivePlayerIds()
for (team in instance.teams) for (p in team) candidates += p.uniqueId
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P2 Badge Preserve assigned player IDs when dissolving voice groups

When a mode clears instance.teams at match start (the new comments explicitly allow this) and participants are removed one by one, GameInstanceEndedEvent is fired only after activePlayerIds has already been emptied, so this candidate set can be empty. In that scenario no online players are moved out of the SimpleVoiceChat groups, leaving them isolated in the old team group after the match/lobby transition. Store the UUIDs assigned in groupsByInstance and use that snapshot during dissolve instead of relying on the already-cleared instance state.

Useful? React with 👍 / 👎.

@EnderDissa EnderDissa closed this May 8, 2026
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