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

Add discord-rpc #2577

Closed
wants to merge 40 commits into from
Closed

Add discord-rpc #2577

wants to merge 40 commits into from

Conversation

Deihim007
Copy link
Contributor

@Deihim007 Deihim007 commented Mar 22, 2022

Based on #2499
Prior discussions/implementations #721, #1031, #1570

This PR adds basic discord-rpc features to the client.
(Other features will be added later on a separate PR)

Todo:

  • Todolist
  • Add discord-rpc
    • Installation script
    • dependencies (rapidjson)
  • State:
    • Main Menu
    • In-game
      • Elapsed time

Maybe:

  • Default game-mode implementation
    • Idle (Custom state)
    • Loading
    • Player count ?
  • Custom Application Id
    • Server-side function
    • Client settings option to disable it (prevent potential abuse)
    • Wiki tutorial
      • How to make an application on discord developer portal
        • Assets template ?

@Lpsd Lpsd mentioned this pull request Mar 29, 2022
@patrikjuvonen patrikjuvonen added the enhancement New feature or request label Apr 2, 2022
@patrikjuvonen patrikjuvonen linked an issue Apr 15, 2022 that may be closed by this pull request
Client/core/CCore.cpp Outdated Show resolved Hide resolved
Client/core/CCore.cpp Outdated Show resolved Hide resolved
Client/core/CCore.cpp Outdated Show resolved Hide resolved
Client/core/CCore.h Outdated Show resolved Hide resolved
Client/core/CCore.h Outdated Show resolved Hide resolved
Client/core/CDiscordRichPresence.h Outdated Show resolved Hide resolved
Client/core/CMainMenu.cpp Outdated Show resolved Hide resolved
Client/sdk/core/CDiscordInterface.h Show resolved Hide resolved
utils/buildactions/install_discord.lua Outdated Show resolved Hide resolved
vendor/discord-rpc/premake5.lua Outdated Show resolved Hide resolved
Deihim007 and others added 8 commits July 28, 2022 00:56
Co-authored-by: Marek Kulik <me@botder.com>
Co-authored-by: Marek Kulik <me@botder.com>
Fix uppercase
@patrikjuvonen patrikjuvonen added the feedback Further information is requested label Dec 30, 2022
@patrikjuvonen patrikjuvonen marked this pull request as draft December 30, 2022 19:59
@Fernando-A-Rocha
Copy link
Contributor

Fernando-A-Rocha commented Feb 16, 2023

I think we can prioritize making customization features for servers to decide what shows on their players' Discord Rich Presence when in-game. Imo, the PR can be merged once this is complete.

Then, allowing servers to integrate custom applications to their servers is something that can be addressed separately.

@github-actions
Copy link
Contributor

This draft pull request is stale because it has been open for at least 90 days with no activity. Please continue on your draft pull request or it will be closed in 30 days automatically.

@github-actions github-actions bot added the stale Inactive for over 90 days, to be closed label Jun 24, 2023
@github-actions github-actions bot closed this Jul 25, 2023
@github-actions
Copy link
Contributor

This draft pull request was closed because it has been marked stale for 30 days with no activity.

@Dutchman101
Copy link
Member

Dutchman101 commented Aug 5, 2023

We need someone to adopt this PR
Removed draft status, as todo list has been fully checkmarked, so that should have been outdated

@Dutchman101 Dutchman101 reopened this Aug 5, 2023
@Dutchman101 Dutchman101 marked this pull request as ready for review August 5, 2023 19:35
znvjder added a commit to znvjder/mtasa-blue that referenced this pull request Aug 28, 2023
znvjder added a commit to znvjder/mtasa-blue that referenced this pull request Aug 28, 2023
Lpsd added a commit that referenced this pull request Oct 11, 2023
* get draft from #2577

* get draft from #2577

* add checkbox to allow connecting with discord-rpc

test case with load data

* safe delete only if used

* fix vars when game is launching

* add function to set appid and small fixes

* added functions to manage assets, small fixes and bugs

* rewriten setAsset, refactor littlebit

* add function to check client connection with discord

* add buttons to rpc

* minor amendments, renaming of functions

* remove update discord-rpc from build

* thats too

* Memory leak fixed, presence updated on app change

* Presence state change, change of time variable

* Fixed unable to change custom state

* Added const to IsDiscordRPCEnabled

* Added missing argument validation

* Exclude vendor files, added premake script

* Re-run actions

* Bump discord-rpc version

* Fix broken discord tag_name

* Bump discord-rpc version

* Revert to 2b42cd9

* Bump discord-rpc version

* Disabled data customization if the app is not own

* Change DEFAULT_APP_ID

* Remove comment

---------

Co-authored-by: Lpsd <40902730+Lpsd@users.noreply.github.com>
@Lpsd
Copy link
Member

Lpsd commented Oct 11, 2023

Superseded by #3167

@Lpsd Lpsd closed this Oct 11, 2023
TracerDS pushed a commit to TracerDS/mtasa-blue that referenced this pull request Feb 2, 2024
* get draft from multitheftauto#2577

* get draft from multitheftauto#2577

* add checkbox to allow connecting with discord-rpc

test case with load data

* safe delete only if used

* fix vars when game is launching

* add function to set appid and small fixes

* added functions to manage assets, small fixes and bugs

* rewriten setAsset, refactor littlebit

* add function to check client connection with discord

* add buttons to rpc

* minor amendments, renaming of functions

* remove update discord-rpc from build

* thats too

* Memory leak fixed, presence updated on app change

* Presence state change, change of time variable

* Fixed unable to change custom state

* Added const to IsDiscordRPCEnabled

* Added missing argument validation

* Exclude vendor files, added premake script

* Re-run actions

* Bump discord-rpc version

* Fix broken discord tag_name

* Bump discord-rpc version

* Revert to 2b42cd9

* Bump discord-rpc version

* Disabled data customization if the app is not own

* Change DEFAULT_APP_ID

* Remove comment

---------

Co-authored-by: Lpsd <40902730+Lpsd@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request feedback Further information is requested stale Inactive for over 90 days, to be closed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add Discord RPC
8 participants