Skip to content

feat: scheduled event exceptions - #1677

Open
kshitijanurag wants to merge 2 commits into
discordjs:mainfrom
kshitijanurag:feat-scheduled-event-exceptions
Open

feat: scheduled event exceptions#1677
kshitijanurag wants to merge 2 commits into
discordjs:mainfrom
kshitijanurag:feat-scheduled-event-exceptions

Conversation

@kshitijanurag

@kshitijanurag kshitijanurag commented Jun 9, 2026

Copy link
Copy Markdown
Contributor

Please describe the changes this PR makes and why it should be merged:
Add support for guild scheduled event exceptions

If applicable, please reference Discord API Docs PRs or commits that influenced this PR:

@kshitijanurag
kshitijanurag requested a review from a team as a code owner June 9, 2026 20:32
@vercel

vercel Bot commented Jun 9, 2026

Copy link
Copy Markdown

@kshitijanurag is attempting to deploy a commit to the discordjs Team on Vercel.

A member of the Team first needs to authorize it.

@kshitijanurag
kshitijanurag force-pushed the feat-scheduled-event-exceptions branch from d18998c to f73beff Compare June 12, 2026 14:14

@almeidx almeidx left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

preliminary

Comment thread rest/v10/index.ts
* - DELETE `/guilds/{guild.id}/scheduled-events/{guildScheduledEvent.id}/exceptions/{exception.id}`
*/
guildScheduledEventException(guildId: Snowflake, guildScheduledEventId: Snowflake, exceptionId?: Snowflake) {
const parts = ['', 'guilds', guildId, 'scheduled-events', guildScheduledEventId];

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Suggested change
const parts = ['', 'guilds', guildId, 'scheduled-events', guildScheduledEventId];
const parts = ['', 'guilds', guildId, 'scheduled-events', guildScheduledEventId, 'exceptions'];

/**
* The new time at when the scheduled event recurrence will start, if applicable
*/
scheduled_start_time: number | null;

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Suggested change
scheduled_start_time: number | null;
scheduled_start_time: string | null;

/**
* The new time at when the scheduled event recurrence will end, if applicable
*/
scheduled_end_time: number | null;

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Suggested change
scheduled_end_time: number | null;
scheduled_end_time: string | null;

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants