Skip to content

Conversation

@foxfirecodes
Copy link
Contributor

@foxfirecodes foxfirecodes commented Aug 10, 2025

Please describe the changes this PR makes and why it should be merged:

a few weeks back i added some new functionality to activities around being able to make state/details/large/small image all have clickable links, as well as allowing users to customize which of name/state/details is used in their status message. this PR adds the corresponding new fields and types to the GatewayActivity type

i wasn't sure whether or not to edit just the v10 types or the other versions. this is my first time contributing to this repo so any guidance is appreciated :3
edit: i saw that changes to v8 and below were disallowed by CI, so i am assuming that i should then be making changes to v9 and v10. if this was wrong plz lmk

related pr that was looking to use these new types: pear-devs/pear-desktop#3692

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

discord/discord-api-docs#7674

@vercel
Copy link

vercel bot commented Aug 10, 2025

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

Name Status Preview Comments Updated (UTC)
discord-api-types ✅ Ready (Inspect) Visit Preview 💬 Add feedback Aug 10, 2025 2:42pm

@foxfirecodes foxfirecodes changed the title feat: add url & status display type fields feat(GatewayActivity): add url & status display type fields Aug 10, 2025
* @see {@link https://discord.com/developers/docs/topics/gateway-events#activity-object-activity-assets}
*/
export type GatewayActivityAssets = Partial<
Record<'large_image' | 'large_text' | 'small_image' | 'small_text', string>
Copy link
Member

Choose a reason for hiding this comment

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

I see this change isn't also in v9, is that intentional?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

woops! that was a mistake. i pushed a fix!

@vladfrangu
Copy link
Member

Also you nailed it otherwise, v9 and v10 are getting the updates and you even got the deno part! 🥹💙

@vladfrangu vladfrangu merged commit 5f9c1e1 into discordjs:main Aug 10, 2025
6 checks passed
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.

Implement new fields for activities

2 participants