Skip to content

feat: notify on chat when new record is established#26

Open
daimonbot wants to merge 1 commit intomainfrom
feature/record-notification-reimpl
Open

feat: notify on chat when new record is established#26
daimonbot wants to merge 1 commit intomainfrom
feature/record-notification-reimpl

Conversation

@daimonbot
Copy link
Collaborator

Closes #7

Implements notifications when a user establishes a new record time for a game in a group chat.

Changes:

  • Added RecordNotificationService that listens to GameSessionRegistrationEvent
  • Added repository method to fetch top 2 sessions by duration
  • Extended GameSessionRegistrationEvent with gameType field
  • Made USER_INTERACTION_NOTIFICATION_ORDER package-private
  • Added comprehensive unit tests

Behavior:

  • Sends notification when first-ever time is recorded
  • Sends notification when previous record is broken (shows both times)
  • Handles users with and without username
  • No notification when time is not a record

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.

[Feature]: Notify on chat when new record is established

1 participant