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

DiscordRPC: Show session time in Discord Rich Presence #10397

Merged
merged 1 commit into from
Dec 23, 2023

Conversation

CookiePLMonster
Copy link
Contributor

Description of Changes

This PR changes the way Discord Rich Presence shows time elapsed. Now instead of updating on every single RPC change, the time only gets reset on:

  • Starting the emulator
  • Starting/stopping the game

Rationale behind Changes

It is more meaningful to show how long the user has been in a specific game than to show how long the current Rich Presence string has been active for.

Suggested Testing Steps

  1. Run any game with RetroAchievements Rich Presence, with Discord Rich Presence enabled.
  2. Perform any actions in-game that change the presence string.
  3. Confirm that the "Time elapsed" display on Discord does not reset when the presence string changes.

Closes #10390

@CookiePLMonster CookiePLMonster marked this pull request as draft December 20, 2023 12:28
@CookiePLMonster CookiePLMonster marked this pull request as ready for review December 20, 2023 17:36
@stenzek stenzek merged commit 0a2e137 into PCSX2:master Dec 23, 2023
12 checks passed
@CookiePLMonster CookiePLMonster deleted the discord-rpc branch December 23, 2023 11:57
@PCSX2 PCSX2 deleted a comment from TheArtemMaps Dec 23, 2023
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.

[Feature Request]: Discord Rich Presence should show the session time as "time elapsed"
5 participants