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

remove poll message #336

Merged
merged 1 commit into from
Dec 21, 2021
Merged

remove poll message #336

merged 1 commit into from
Dec 21, 2021

Conversation

modernNeo
Copy link
Member

Description

removing the poll command when the user invokes it so the channel isn't littered with command calls

PR Checklist

These are the things you need to ensure are covered in your PR, otherwise the CODEOWNERS will not approve your PR, not matter how much you ping them to do so on the Discord

  1. The description in the PR is a fair representation of what the PR is about.
  2. The PR is fixing one thing and one thing only.
  3. Logging. if you have N variables initialzed/used in your function, you should print all of them out to the log using logging module at least once or have a good reason why you arent.
  4. If your PR is doing something like adding a new line or removing a new line, CODEOWNERS reserve the right to ask that you undo that change unless it was for a specific reason.
  5. If you are adding a new command....document. Document the following things on help.json and the README.md
    1. The purpose of the command
    2. If the argument is called with any arguments.
      1. If it is called with any arguments, please either provide a good enough explanation of the arg that a user can tell what it will do before using the command. adding an example of how to call it with the args is not necessary but good practice.
  6. If you are making a new Class of commands, add the class to bot.json following the convention already there.
  7. Evidence of Testing. This one needs to be completed after the PR is opened. At that point, you will go on the channel on the CSSS Wall-E Test Server that was automatically created when the PR was opened and then test the following functionality. Once you had done so, you can leave a comment on the PR stating that you had done the necessary testing.
  8. Please provide ways to test whatever you just modified on the bot in the Test Cases section so that future PRs can be tested to ensure they dont break your code when merging to master

@modernNeo modernNeo requested a review from a team as a code owner December 21, 2021 18:27
@modernNeo modernNeo merged commit b4ef973 into master Dec 21, 2021
@modernNeo modernNeo deleted the remove_poll_message branch December 21, 2021 18:29
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