Skip to content

feat(moderation): handle ban and kick notifications - #571

Open
tintinhamans wants to merge 1 commit into
GeneralsOnlineDevelopmentTeam:mainfrom
tintinhamans:arctic/feature/moderation-dialogs
Open

feat(moderation): handle ban and kick notifications#571
tintinhamans wants to merge 1 commit into
GeneralsOnlineDevelopmentTeam:mainfrom
tintinhamans:arctic/feature/moderation-dialogs

Conversation

@tintinhamans

@tintinhamans tintinhamans commented Aug 21, 2026

Copy link
Copy Markdown
  • Handle websocket moderation action message 46 for bans and kicks.
  • Display the server-provided moderation reason.
  • Present consistent Banned and Kicked dialogs with Discord and Close actions.
  • Direct players to Discord for more information or support.
  • Tear down the online session and return the player to the main menu.
  • Handle bans detected during login and background token refresh.
  • Normalize moderation reasons and safely reject malformed UTF-8.

The accompanying server PR sends a moderation message before disconnecting a banned or kicked player. This lets the client distinguish moderation actions from connection issues and also provides them with a Discord link for questions or appeals.

Unpatched clients will not display the moderation reason, but the server still disconnects them/denies login as before.

Related: GeneralsOnlineDevelopmentTeam/Services#49

@github-actions

Copy link
Copy Markdown

⚠️ Title/Commit Validation Failed

Invalid PR title:

  • feat(moderation): handle ban and kick notifications

Invalid commit messages:

  • feat(moderation): handle ban and kick notifications
    PR titles and commit messages must follow conventional commits format:
type: Description
type(scope): Description

Allowed types: bugfix, build, chore, ci, docs, fix, feat, perf, refactor, revert, style, test, tweak, unify

See CONTRIBUTING.md for details.

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