Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Owner notification system #6

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

MilleniumModsSources
Copy link
Contributor

Further improvements to the notifyOwner system.

  • Improved the notification message when running Administrator commands by adding an extra option to customize the notifyCommandMessage code.
  • Add the notifyCommandMessage for notifications when an Administrator command is executed.

How it works:
If a command that requires Administrator permissions is executed, and the notifyOwner option is enabled, the notifyCommandMessage will be executed. However, if the notifyCommandMessage option is missing, the bot will simply send the default message which is set in /src/Command.js (Already tested properly, so hopefully this will work right away unlike the last PullRequest 😅)

Improved the notification message when running Administrator commands by adding an extra option to customize the event code
Add the message for notifications when an Administrator command is executed.

How it works:
If a command that requires Administrator permissions is executed, and the notifyOwner option is enabled, the notifyCommandMessage will be executed. However, if the notifyCommandMessage option is missing, the bot will simply send the default message which is set in /src/Command.js (Already tested properly, so hopefully this will work right away unlike the last PullRequest 😅)
@MilleniumModsSources
Copy link
Contributor Author

bump

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant