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: Auto Moderation #7938

Merged
merged 102 commits into from
Nov 19, 2022
Merged

feat: Auto Moderation #7938

merged 102 commits into from
Nov 19, 2022

Conversation

Jiralite
Copy link
Member

@Jiralite Jiralite commented May 17, 2022 •
edited by ghost
Loading

🤖 Generated by Copilot at e4c1f63

Summary

🛡️🚨🧩

This pull request implements auto moderation functionality for discord.js, by adding new classes, events, actions, handlers, and types related to auto moderation rules and actions. It also updates the existing structures and constants to support the new features, and modifies the bug report template to include the new events.

This pull request adds some new features
To handle auto moderation rules
With new classes and events
And some API types and intents
The client can now deal with the tools

Walkthrough

  • Add support for auto moderation rules and actions to the client, guild, audit logs and websocket classes (link, link, link, link, link, link, link, link, link, link, link, link, link, link, link, link, link, link, link, link, link, link, link, link, link, link)
  • Export the AutoModerationRuleManager, AutoModerationActionExecution and AutoModerationRule classes from the index.js file (link, link)
  • Add two new event types to the bug report template for auto moderation issues (link)
  • Import the AutoModerationRule class from the structures folder to the AutoModerationActionExecution and GuildAuditLogsEntry classes (link, link)
  • Import the AutoModerationRuleManager class from the managers folder to the Guild class (link)
  • Add external typedefs for the auto moderation enums from the discord-api-types documentation to the APITypes.js file (link)

@monbrey
Copy link
Member

monbrey commented May 18, 2022

Are we expecting far more documentation upstream? Seems like there's a lot missing from the Discord implementation like API endpoints, something on the Guild structure to store them etc

@imranbarbhuiya
Copy link
Contributor

Are we expecting far more documentation upstream? Seems like there's a lot missing from the Discord implementation like API endpoints, something on the Guild structure to store them etc

discord/discord-api-docs#4860 (comment)

@iCrawl iCrawl added this to the discord.js v14 milestone May 18, 2022
@BenPatersonxx

This comment was marked as spam.

Sign in to view
@vercel
Copy link

vercel bot commented Jun 8, 2022 •
edited
Loading

The latest updates on your projects. Learn more about Vercel for Git ↗︎

2 Ignored Deployments
Name Status Preview Updated
discord-js ⬜️ Ignored (Inspect) Nov 19, 2022 at 9:59PM (UTC)
discord-js-guide ⬜️ Ignored (Inspect) Nov 19, 2022 at 9:59PM (UTC)

@vercel vercel bot temporarily deployed to Preview June 8, 2022 18:34 Inactive
@vercel vercel bot temporarily deployed to Preview June 8, 2022 23:10 Inactive
Mateo-tem
Mateo-tem suggested changes Jun 11, 2022
View reviewed changes
packages/discord.js/typings/index.d.ts Outdated Show resolved Hide resolved
packages/discord.js/typings/index.d.ts Outdated Show resolved Hide resolved
@almeidx almeidx mentioned this pull request Jun 16, 2022
Closed
ArjunSharda
ArjunSharda suggested changes Jun 17, 2022
View reviewed changes
Copy link

@ArjunSharda ArjunSharda left a comment

Choose a reason for hiding this comment

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

I think the changes I put should be good, as I don't think the spaces are necessary.

packages/discord.js/src/client/actions/AutoModerationRuleDelete.js Show resolved Hide resolved
packages/discord.js/src/client/actions/AutoModerationRuleUpdate.js Show resolved Hide resolved
ImRodry
ImRodry suggested changes Jun 17, 2022
View reviewed changes
Copy link
Contributor

@ImRodry ImRodry left a comment

Choose a reason for hiding this comment

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

Seeing as some endpoints are not available in most (if not all) guilds it would be nice to add a warning for that

.github/ISSUE_TEMPLATE/bug_report.yml Show resolved Hide resolved
packages/discord.js/src/client/actions/AutoModerationActionExecution.js Outdated Show resolved Hide resolved
packages/discord.js/src/managers/AutoModerationRuleManager.js Outdated Show resolved Hide resolved
packages/discord.js/src/managers/AutoModerationRuleManager.js Outdated Show resolved Hide resolved
packages/discord.js/src/managers/AutoModerationRuleManager.js Outdated Show resolved Hide resolved
packages/discord.js/src/structures/AutoModerationActionExecution.js Outdated Show resolved Hide resolved
packages/discord.js/src/structures/AutoModerationActionExecution.js Show resolved Hide resolved
packages/discord.js/src/structures/AutoModerationActionExecution.js Outdated Show resolved Hide resolved
packages/discord.js/src/structures/AutoModerationActionExecution.js Show resolved Hide resolved
packages/discord.js/src/structures/AutoModerationRule.js Outdated Show resolved Hide resolved
@Jiralite Jiralite changed the title feat: AutoMod feat: Auto Moderation Jun 27, 2022
imranbarbhuiya
imranbarbhuiya reviewed Jul 3, 2022
View reviewed changes
.husky/pre-commit Outdated Show resolved Hide resolved
@vercel vercel bot temporarily deployed to Preview July 9, 2022 19:47 Inactive
@vercel vercel bot temporarily deployed to Preview July 25, 2022 14:25 Inactive
@vercel vercel bot temporarily deployed to Preview July 25, 2022 14:38 Inactive
@vercel vercel bot temporarily deployed to Preview July 25, 2022 15:32 Inactive
@vercel vercel bot temporarily deployed to Preview July 26, 2022 12:16 Inactive
@vercel vercel bot temporarily deployed to Preview July 27, 2022 11:01 Inactive
@vercel vercel bot temporarily deployed to Preview July 27, 2022 20:42 Inactive
@iCrawl iCrawl added the blocked label Jul 29, 2022
Jiralite and others added 15 commits November 4, 2022 23:08
@Jiralite Jiralite requested a review from vladfrangu November 19, 2022 21:15
@kodiakhq kodiakhq bot merged commit fd4ba5e into discordjs:main Nov 19, 2022
@Jiralite Jiralite deleted the feat/auto-moderation branch November 19, 2022 22:09
@SuperchupuDev SuperchupuDev mentioned this pull request Nov 20, 2022
Merged
@jaw0r3k jaw0r3k mentioned this pull request Nov 30, 2022
Merged
@Jiralite Jiralite mentioned this pull request Apr 7, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@imranbarbhuiya imranbarbhuiya imranbarbhuiya left review comments

@almeidx almeidx almeidx left review comments

@ImRodry ImRodry ImRodry requested changes

@Qjuh Qjuh Qjuh left review comments

@didinele didinele didinele left review comments

@Eejit43 Eejit43 Eejit43 left review comments

@suneettipirneni suneettipirneni suneettipirneni requested changes

@tristancamejo tristancamejo tristancamejo left review comments

@NotSugden NotSugden NotSugden left review comments

@Mateo-tem Mateo-tem Mateo-tem requested changes

@ArjunSharda ArjunSharda ArjunSharda requested changes

@GoldenAngel2 GoldenAngel2 GoldenAngel2 requested changes

@axisiscool axisiscool axisiscool requested changes

@kyranet kyranet kyranet approved these changes

@SpaceEEC SpaceEEC SpaceEEC approved these changes

@vladfrangu vladfrangu vladfrangu approved these changes

@iCrawl iCrawl Awaiting requested review from iCrawl iCrawl is a code owner

Assignees
No one assigned
Labels
api support packages:discord.js semver:minor
Projects
discord.js
Archived in project
Milestone
discord.js 14.7
Development

Successfully merging this pull request may close these issues.