Skip to content

feat: Introduce IGuildMusicService for guild-specific music management#106

Merged
standleypg merged 1 commit into
masterfrom
feat/105-multi-guilds-support
Jul 12, 2026
Merged

feat: Introduce IGuildMusicService for guild-specific music management#106
standleypg merged 1 commit into
masterfrom
feat/105-multi-guilds-support

Conversation

@standleypg

Copy link
Copy Markdown
Collaborator
  • Added IGuildMusicService interface to handle music playback per guild.
  • Implemented GuildPlayer and GuildPlayerManager to manage music queues and playback for multiple guilds concurrently.
  • Updated AdminCommands and MusicActionCommands to utilize the new IGuildMusicService for handling music actions.
  • Removed IMusicPlayerController as it was replaced by the new guild-specific service.
  • Enhanced event handling for Skip and Stop events to be guild-scoped.
  • Updated dependency injection to register the new services and components.
  • Added unit tests for GuildPlayerManager and GuildPlayer to ensure correct functionality.
    Multiple guilds support #105

- Added IGuildMusicService interface to handle music playback per guild.
- Implemented GuildPlayer and GuildPlayerManager to manage music queues and playback for multiple guilds concurrently.
- Updated AdminCommands and MusicActionCommands to utilize the new IGuildMusicService for handling music actions.
- Removed IMusicPlayerController as it was replaced by the new guild-specific service.
- Enhanced event handling for Skip and Stop events to be guild-scoped.
- Updated dependency injection to register the new services and components.
- Added unit tests for GuildPlayerManager and GuildPlayer to ensure correct functionality.
@standleypg
standleypg merged commit f206d18 into master Jul 12, 2026
1 check passed
@standleypg
standleypg deleted the feat/105-multi-guilds-support branch July 12, 2026 17:40
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