Skip to content

Conversation

@fxzzi
Copy link
Contributor

@fxzzi fxzzi commented Aug 19, 2025

As noticed by the spotify rpc integration a while ago, it recently changed from showing "Listening to Spotify", to "Listening to <ArtistName>". This was then enabled for regular rpc here: https://discord.com/developers/docs/change-log#clickable-links-and-customizable-statuses-in-rich-presence

Then implemented in discord-presence here: jewlexx/discord-presence#143

This PR simply exposes an option to switch between name (default), state (which in my case is the artist name), and details (which in my case is the song name)

image image

Maybe the default should be changed? I've left it as name for now which means it'll just show "Listening to music" with the default app id.

image

@JakeStanger
Copy link
Owner

Oh awesome, I had no idea that had been added. Thanks for this!

Maybe the default should be changed?

I think so. The fact it says 'music' was really just because there wasn't anything better to put there.

@fxzzi
Copy link
Contributor Author

fxzzi commented Aug 19, 2025

sure, I went ahead and changed the default to "state", which by default is the artist name. Updated the readme slightly to better explain what the option does.

Also, formatted the rest of the README. let me know if you want me to drop that.

@JakeStanger
Copy link
Owner

Happy for that to be kept. Can you just update your commits to follow conventional commit format though please so that the readme entries generate correctly?

feat: allow configuring status display
style: format readme

@fxzzi
Copy link
Contributor Author

fxzzi commented Aug 19, 2025

all fixed up for ya :)

@fxzzi fxzzi changed the title allow configuring status display feat: allow configuring status display Aug 19, 2025
@JakeStanger JakeStanger merged commit 497bab9 into JakeStanger:master Aug 19, 2025
1 check passed
@fxzzi fxzzi deleted the status-display branch August 19, 2025 23:56
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.

2 participants