Skip to content
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

adminchannel: restore op/voice commands #1498

Merged
merged 1 commit into from
May 12, 2019
Merged

Conversation

dgw
Copy link
Member

@dgw dgw commented Mar 11, 2019

Essentially reverts commit 902e715, but restores edited code that meets Sopel's current standards (using bot.channels; providing error messages for @require_* decorators).

Closes #1350

@dgw dgw added the Feature label Mar 11, 2019
@dgw dgw added this to the 7.0.0 milestone Mar 11, 2019
@dgw dgw mentioned this pull request Mar 18, 2019
Copy link
Contributor

@Exirel Exirel left a comment

Choose a reason for hiding this comment

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

LGTM

Essentially reverts 902e715, but with
code that meets Sopel's current code standards (using `bot.channels`;
providing error messages for `@require_*` decorators).

Closes #1350
@dgw dgw force-pushed the 1350-restore-adminchannel-cmds branch from 8fa1a27 to 5cbb7e6 Compare May 12, 2019 03:30
@dgw dgw merged commit f8f2eee into master May 12, 2019
@dgw dgw deleted the 1350-restore-adminchannel-cmds branch May 12, 2019 03:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

adminchannel: Restore op/voice commands
2 participants