-
Notifications
You must be signed in to change notification settings - Fork 44
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
#42: Command /autopause
#98
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Super clean, love it 👌
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great, I like your refactoring of the /events dir.
A behaviour went against my intuition: skipping an autopaused song also pauses the next one. From what I've investigated, there's no simple way to check whether the track was paused before being skipped. Let's revisit this later if after using it for a while we find it that frustrating.
/autopause
as a toggle, similar to/repeat
.Make use of serenity's context cache to store
GuildSettings
such as theautopause
flag. @aquelemiguel will be using it for other Guild-related cache.Tested the edge cases of skipping and seeking to the end of the track with autopause on.