Skip to content

feat: add match map pause notifications for admins and tournament org…#101

Merged
lukepolo merged 5 commits intomainfrom
feat/match-tournament-discord-web-changes
Mar 2, 2026
Merged

feat: add match map pause notifications for admins and tournament org…#101
lukepolo merged 5 commits intomainfrom
feat/match-tournament-discord-web-changes

Conversation

@Flegma
Copy link
Copy Markdown
Contributor

@Flegma Flegma commented Mar 2, 2026

…anizers

Flegma added 2 commits March 2, 2026 17:04
Replace plain text Discord messages with embed payloads that have
colored sidebars matching the web app's badge colors: green for
live/finished/veto states, red for canceled/paused/error states,
and gray for all other statuses.
Convert send() Discord payload from plain text to embeds with
color-coded severity (RED for errors/offline, GREEN for online,
GRAY default for informational).
Comment thread src/notifications/notifications.service.ts Outdated
Comment thread src/notifications/notifications.service.ts Outdated
Comment thread src/notifications/notifications.service.ts Outdated
Comment thread package-lock.json Outdated
Move STATUS_LABELS, DISCORD_COLORS, and STATUS_COLORS to
notifications/utilities/constants.ts with proper e_match_status_enum
typing. Simplify tournament bracket extraction using .at(0)?.stage.tournament
and early return pattern. Type sendMatchStatusNotification params.
Remove package-lock.json (project uses yarn).
@Flegma
Copy link
Copy Markdown
Contributor Author

Flegma commented Mar 2, 2026

Check latest commit 8ab5e89 if i've implemented all changes you've suggested @lukepolo

Comment thread src/game-server-node/jobs/CheckServerPluginVersions.ts Outdated
Decouple consumers from NotificationsService for the DISCORD_COLORS
constant by importing directly from notifications/utilities/constants.
@lukepolo lukepolo merged commit b93d4e7 into main Mar 2, 2026
1 check passed
@lukepolo lukepolo deleted the feat/match-tournament-discord-web-changes branch March 2, 2026 17:35
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.

2 participants