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.

Sign up for GitHub

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

Jump to bottom

feat(AutoModeration): Support Auto Moderation #1390

Closed
wants to merge 11 commits into from
Closed

feat(AutoModeration): Support Auto Moderation #1390

wants to merge 11 commits into from

Conversation

DonovanDMC
Copy link
Contributor

No description provided.

@DonovanDMC DonovanDMC mentioned this pull request Jun 18, 2022
Closed
@DonovanDMC DonovanDMC requested a review from bsian03 June 18, 2022 02:37
DonovanDMC added a commit to DonovanArchive/ErisPRUpdateBot that referenced this pull request Jun 18, 2022
@DonovanDMC
Add http://abalabahaha/eris#1390
646d433
DonovanDMC added a commit to DonovanArchive/ErisPRUpdateBot that referenced this pull request Jun 18, 2022
@DonovanDMC
Add abalabahaha/eris#1390
1fe5a08
@abalabahaha abalabahaha added this to the 0.17.x milestone Jun 29, 2022
frobinsonj
frobinsonj reviewed Jul 7, 2022
View reviewed changes
Copy link
Contributor

@frobinsonj frobinsonj left a comment

Choose a reason for hiding this comment

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

Create/Modify/Delete Auto Moderation Rule endpoints support the X-Audit-Log-Reason header.

It should be possible to specify a reason. Not sure whether that should be put in the options or have a separate parameter.

@DonovanDMC
Copy link
Contributor Author

Create/Modify/Delete Auto Moderation Rule endpoints support the X-Audit-Log-Reason header.

It should be possible to specify a reason. Not sure whether that should be put in the options or have a separate parameter.

Done, as far as I'm aware we're actively moving to providing reasons in the options object if possible

@TheEssem TheEssem mentioned this pull request Jul 20, 2022
Merged
HeadTriXz
HeadTriXz suggested changes Aug 18, 2022
View reviewed changes
lib/gateway/Shard.js Show resolved Hide resolved
index.d.ts Show resolved Hide resolved
HeadTriXz
HeadTriXz suggested changes Aug 19, 2022
View reviewed changes
index.d.ts Outdated Show resolved Hide resolved
@DonovanDMC @HeadTriXz
Update index.d.ts
b82b266 
Co-authored-by: HeadTriXz <[email protected]>
@DonovanDMC DonovanDMC self-assigned this Aug 20, 2022
HeadTriXz
HeadTriXz suggested changes Aug 20, 2022
View reviewed changes
index.d.ts Show resolved Hide resolved
@DonovanDMC DonovanDMC closed this Aug 29, 2022
@DonovanDMC DonovanDMC deleted the automod branch August 29, 2022 00:30
@bsian03 bsian03 mentioned this pull request Nov 18, 2022
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@frobinsonj frobinsonj frobinsonj left review comments

@coolcalcacol coolcalcacol coolcalcacol left review comments

@HeadTriXz HeadTriXz HeadTriXz requested changes

@bsian03 bsian03 Awaiting requested review from bsian03

Assignees

@DonovanDMC DonovanDMC

Labels
Discord
Projects
None yet
Milestone
0.17.3
Development

Successfully merging this pull request may close these issues.
6 participants
@DonovanDMC @frobinsonj @bsian03 @coolcalcacol @HeadTriXz @abalabahaha