Skip to content

Add Discord Solve/First Blood Notifications, Nickname Synchronization, and 10-Character Name Limit#63

Merged
yulmwu merged 3 commits into
mainfrom
feat/0703
Jul 4, 2026
Merged

Add Discord Solve/First Blood Notifications, Nickname Synchronization, and 10-Character Name Limit#63
yulmwu merged 3 commits into
mainfrom
feat/0703

Conversation

@yulmwu

@yulmwu yulmwu commented Jul 3, 2026

Copy link
Copy Markdown
Member

This PR adds Discord notifications for challenge solves. Messages are sent to the channel specified by DISCORD_ANNOUNCE_CHANNEL_ID, with First Blood solves receiving special emphasis.

Example:

🩸 First Blood! 청소년부_B — kim6wol0 solved member_directory test!
청소년부_A — KimJunYoung solved member_directory test

In addition, the Invite Bot now automatically sets a member's guild nickname to the format:

<Division>_<TeamName>_<Username>

If a user later updates their profile information, the bot automatically synchronizes the guild nickname to reflect the updated values.

Since Discord limits nicknames to 32 characters, the division name, team name, and username are each now limited to 10 characters to ensure the generated nickname remains within the limit.

For more details, please refer to the changes included in this PR.

Note

In a future update, different Discord roles and solve announcement channels will be configurable on a per-division basis.

@yulmwu yulmwu added the enhancement New feature or request label Jul 3, 2026

@yulmwu yulmwu left a comment

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

It looks like the REST API documentation needs to be updated to reflect these changes.

@codecov

codecov Bot commented Jul 3, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 78.57143% with 18 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
internal/service/discord_service.go 64.58% 10 Missing and 7 partials ⚠️
internal/repo/submission_repo.go 93.33% 1 Missing ⚠️

📢 Thoughts on this report? Let us know!

Comment thread internal/http/handlers/handler.go
Comment thread internal/http/handlers/handler.go Outdated
Comment thread internal/service/discord_service.go
Comment thread internal/service/discord_service.go
Comment thread internal/http/handlers/handler.go Outdated
Comment thread internal/service/discord_service.go
@yulmwu
yulmwu merged commit e6a811b into main Jul 4, 2026
2 checks passed
@yulmwu
yulmwu deleted the feat/0703 branch July 4, 2026 03:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant