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

Update avatar decoration docs #6464

Merged
merged 5 commits into from
May 14, 2024
Merged

Conversation

advaith1
Copy link
Contributor

The field and CDN endpoint for avatar decorations have changed

@Lulalaby
Copy link
Contributor

Tables my beloved!

@Lulalaby
Copy link
Contributor

We changed it to this table formatting in 9078f03 🙈

"public_flags": 64
"public_flags": 64,
"avatar_decoration_data": {
"sku_id": "1144058844004233369",
Copy link
Contributor

Choose a reason for hiding this comment

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

What is this sku_id used for?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It identifies the actual decoration - sometimes they edit the image so there are different assets corresponding to the same decoration but the sku id will be the same for everyone. I think there is currently no way for bots to fetch additional information for an sku though.

Copy link
Contributor

@shaydewael shaydewael left a comment

Choose a reason for hiding this comment

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

annoying nit re: removing "the" before "id", but lgtm

docs/resources/User.md Outdated Show resolved Hide resolved
@shaydewael shaydewael self-assigned this May 14, 2024
@shaydewael shaydewael merged commit c583bb1 into discord:main May 14, 2024
shaydewael added a commit to Jupith/discord-api-docs that referenced this pull request May 14, 2024
* Update avatar decoration docs

* add to example user

* oh is this the standard formatting now

* add guild member avatar decoration field

* Update docs/resources/User.md

---------

Co-authored-by: shay <swdewael@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants