Feat: auto-message on moderation message delete #247
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* Set up script to deploy new era commands * Create 'Delete Message' context menu command * Create ModerationMessageDelete service & utils for threads * Fix points tests: botEngine.js wasn't mocked * Update error handling for the case where can't send DMs to users
01zulfi
force-pushed
the
feat/message-delete
branch
from
a651465
to
9aa3f6b
Compare
zachmmeyer
approved these changes
Sep 16, 2022
3 tasks
|
@01zulfi I'll need to look into it more over the next couple days or so before I can really say what it will take to test this. I won't block this PR for the tests so I've created #248 as a followup. If at all possible I would like to see it addressed before any more features are added in after this PR. Additionally, if you could create some followup issues for any tests needed for other recently added features that didn't include tests I would appreciate that. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Complete the following REQUIRED checkboxes:
location of change: brief description of changeformat, e.g.Callbacks command: Update verbiageComplete the following checkboxes ONLY IF they are applicable to your PR. You can complete them later if they are not currently applicable:
1. Because:
Automating moderation actions is handy.
2. This PR:
Behaviour:
To Be Decided:
3. Additional Information:
n/a