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

🤖 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
@Jiralite Jiralite force-pushed the feat/auto-moderation branch 2 times, most recently from 7e50773 to a4ec0f8 Compare May 19, 2022 16:48
@BenPatersonxx

This comment was marked as spam.

Sign in to view
@vercel
Copy link

vercel bot commented Jun 8, 2022 •
edited

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
@Jiralite Jiralite force-pushed the feat/auto-moderation branch 2 times, most recently from 6510b42 to d302d63 Compare June 13, 2022 19:52
@almeidx almeidx mentioned this pull request Jun 16, 2022
Closed
@Jiralite Jiralite force-pushed the feat/auto-moderation branch 2 times, most recently from d6a7f5e to 9cc9a45 Compare June 17, 2022 02:56
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 force-pushed the feat/auto-moderation branch 6 times, most recently from 5a33f1e to df9734e Compare June 24, 2022 09:04
@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

@didinele didinele didinele left review comments

@NotSugden NotSugden NotSugden left review comments

@ImRodry ImRodry ImRodry requested changes

@almeidx almeidx almeidx left review comments

@tristancamejo tristancamejo tristancamejo left review comments

@imranbarbhuiya imranbarbhuiya imranbarbhuiya left review comments

@Qjuh Qjuh Qjuh left review comments

@Eejit43 Eejit43 Eejit43 left review comments

@suneettipirneni suneettipirneni suneettipirneni requested changes

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

@GoldenAngel2 GoldenAngel2 GoldenAngel2 requested changes

@axisiscool axisiscool axisiscool requested changes

@ArjunSharda ArjunSharda ArjunSharda requested changes

@vladfrangu vladfrangu vladfrangu approved these changes

@SpaceEEC SpaceEEC SpaceEEC approved these changes

@kyranet kyranet kyranet 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.

None yet