Skip to content

Conversation

@almeidx
Copy link
Member

@almeidx almeidx commented Jun 14, 2025

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

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

@almeidx almeidx requested review from a team as code owners June 14, 2025 15:28
@vercel
Copy link

vercel bot commented Jun 14, 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 Jun 14, 2025 3:57pm

/**
* Path to the nameplate asset
*
* @example `nameplates/nameplates/twilight/`
Copy link
Member

Choose a reason for hiding this comment

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

Lets have a CDN route function thing for nameplates 👁️

Copy link
Member Author

@almeidx almeidx Jun 20, 2025

Choose a reason for hiding this comment

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

I didn't add it because its not documented and because the structure of it is not exactly clear.
From my testing, nameplates can be either prefixed with nameplates/nameplates/ or nameplates/nameplates_v2/..

There is also a static image static.png, and the video asset.webm

Copy link
Member

Choose a reason for hiding this comment

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

Eugh, they really did nameplates dirty huh.. well w/e

@vladfrangu vladfrangu merged commit 77cb327 into main Jun 20, 2025
6 checks passed
@vladfrangu vladfrangu deleted the feat/nameplates branch June 20, 2025 10:57
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.

4 participants