Skip to content

implement discord status types#1037

Merged
jeffvli merged 2 commits into
jeffvli:developmentfrom
vimaexd:discord-displaytype
Aug 28, 2025
Merged

implement discord status types#1037
jeffvli merged 2 commits into
jeffvli:developmentfrom
vimaexd:discord-displaytype

Conversation

@vimaexd

@vimaexd vimaexd commented Aug 1, 2025

Copy link
Copy Markdown
Contributor

this pr introduces an option to change what is displayed in the discord status: "Feishin", the song's name or the song's artist

20250801_164518 20250801_164602 20250801_164633

(for artist name, the "By" was dropped from the presence data as "Listening to By Vylet Pony" does not look very good - this also matches implemention of other music apps such as Spotify)

@vercel

vercel Bot commented Aug 1, 2025

Copy link
Copy Markdown

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

Name Status Preview Comments Updated (UTC)
feishin ❌ Failed (Inspect) Aug 1, 2025 3:51pm

@pyxfluff

pyxfluff commented Aug 6, 2025

Copy link
Copy Markdown
Contributor

Is this working on vanilla discord? I've only ever seen it work on arRPC-enabled clients like Vesktop.

@vimaexd

vimaexd commented Aug 6, 2025

Copy link
Copy Markdown
Contributor Author

Is this working on vanilla discord? I've only ever seen it work on arRPC-enabled clients like Vesktop.

yes, status_display_type is part of the sdk now:
https://discord.com/developers/docs/events/gateway-events#activity-object-status-display-types

@Lyall-A

Lyall-A commented Aug 12, 2025

Copy link
Copy Markdown
Contributor

Can album name be added? would be pretty nice

@vimaexd

vimaexd commented Aug 12, 2025

Copy link
Copy Markdown
Contributor Author

Can album name be added? would be pretty nice

discord only supports using the Details (first line, song name) or State (second line, artist name) part of the presence data in the status - album name is stored by using the large_image_text in what i can only assume is a hack on their existing data structure

to support album in presence, either the presence data needs to be shuffled around or discord would need to support using large_image_text for status display

@Lyall-A

Lyall-A commented Aug 12, 2025

Copy link
Copy Markdown
Contributor

ahhh alright

@jeffvli jeffvli merged commit 352a8e7 into jeffvli:development Aug 28, 2025
5 checks passed
@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Dec 27, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants