Skip to content

Conversation

@Huntifer-RotMG
Copy link
Contributor

ViBot [8.11.0]

Changelog

Features

  • ;vote now has a slash command
  • The buttons on ;vote will now work across a bot restart

Changes

  • We can now use role arguments in slash commands
  • Restart-safe commands can now mutate the state

Definitely open to design feedback on this, my focus was on adding new features and less on perfecting the style/design of this function. That said, I do this this is pretty solid.

…ommandparser upgrades; and add vote slash command
@Huntifer-RotMG Huntifer-RotMG requested a review from a team as a code owner January 30, 2024 02:55
Copy link

@ghost ghost left a comment

Choose a reason for hiding this comment

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

mostly looks good to me, i would include a change to findChannel in extentions.js used on line 70 or catch this edge case in vote.js. typing the name of a channel that shares the name of a category can load the category into the configuration and therefore errors when sending the vote.

@Huntifer-RotMG
Copy link
Contributor Author

Closing in favor of #751

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