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.

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

Only update name of GuildScheduledEvent if found in data #9834

Closed
wants to merge 1 commit into from
Closed

Only update name of GuildScheduledEvent if found in data #9834

wants to merge 1 commit into from

Conversation

xgedev
Copy link

@xgedev xgedev commented Sep 14, 2023

Please describe the changes this PR makes and why it should be merged:
When e. g. the event "guildScheduledEventUserAdd" is fired, it runs the _patch function of the GuildScheduledEvent object with just the data parameters "id" and "guild_id". Without checking if "name" is in the data object, the name is set to "undefined" when a user subscribes to the event and no longer retrievable unless one fetches the event again.

  • Code changes have been tested against the Discord API, or there are no code changes
  • I know how to update typings and have done so, or typings don't need updating

When e. g. the event "guildScheduledEventUserAdd" is fired, it runs the _patch function of the GuildScheduledEvent object with just the data parameters "id" and "guild_id".
Without checking if "name" is in the data object, the name is set to "undefined" when a user subscribes to the event and no longer retrievable unless one fetches the event again.
@xgedev xgedev requested a review from a team as a code owner September 14, 2023 10:03
@vercel
Copy link

vercel bot commented Sep 14, 2023

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

2 Ignored Deployments
Name Status Preview Comments Updated (UTC)
discord-js ⬜️ Ignored (Inspect) Visit Preview Sep 14, 2023 10:03am
discord-js-guide ⬜️ Ignored (Inspect) Visit Preview Sep 14, 2023 10:03am

@github-actions
Copy link

github-actions bot commented Sep 14, 2023

⚡️ Lighthouse report for the changes in this PR:

Category Score
🟠 Performance 77
🟢 Accessibility 96
🟢 Best practices 100
🟠 SEO 67
🔴 PWA 33

Lighthouse ran on https://discord-js-guide-git-fork-xgedev-patch-1-discordjs.vercel.app/

@Jiralite
Copy link
Member

Duplicate of #9802.

@Jiralite Jiralite closed this Sep 14, 2023
@xgedev xgedev deleted the patch-1 branch September 14, 2023 10:11
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