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
feat: add support for GuildScheduledEvent #6493
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I know this is still a draft, these are just some notes for the future mostly
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some things I noticed after going through this PR again
I also think you should change Invite#INVITES_PATTERN to /discord(?:(?:app)?\.com\/invite|\.gg(?:\/invite)?)\/([\w-]{2,255})$/gi
(added the $ at the end to match the end of the string) and add a new property under GuildEvent called EVENTS_PATTERN with the following regex: /discord(?:(?:app)?\.com\/invite|\.gg(?:\/invite)?)\/[\w-]{2,255}\?event=([0-9]{17,19})/gi
0619667
to
ccc1a17
Compare
This is ready for review |
Co-authored-by: Jiralite <33201955+Jiralite@users.noreply.github.com>
Co-authored-by: Jiralite <33201955+Jiralite@users.noreply.github.com>
Co-authored-by: Antonio Román <kyradiscord@gmail.com>
Co-authored-by: Vlad Frangu <kingdgrizzle@gmail.com>
Co-authored-by: SpaceEEC <spaceeec@yahoo.com>
Co-authored-by: Antonio Román <kyradiscord@gmail.com>
e38bd83
to
b5cfe27
Compare
Please describe the changes this PR makes and why it should be merged:
Adds support for
GuildScheduledEvent
:Todo:
GuildScheduledEvent
classGuildScheduledEvent
classGet
List of guild events in a guildPost
Create a guild eventGet
A guild event by idDelete
Delete a guild eventPatch
Modify a guild eventGuildEventManager
MANAGE_EVENTS
permissionGuildScheduledEvent
related audit logsStatus and versioning classification: