JustYuuto / deezer-discord-rpc

A Discord RPC for Deezer
Mozilla Public License 2.0
55 stars 10 forks source link

Song not display on Discord #65

Closed BlocBlocTheDev closed 1 month ago

BlocBlocTheDev commented 2 months ago

When we listen a music, discord don't display on the profile :

image

image

Same problem in "Playing" mod and "Listening" mod

Sorry for english, i'm a French guy... :/

JustYuuto commented 2 months ago

Je suis français aussi :)

Essaye avec ça : https://github.com/JustYuuto/deezer-discord-rpc/wiki/Troubleshooting

BlocBlocTheDev commented 2 months ago

Re !

Malheureusement le problème persiste, même après avoir tenter de suivre les consigne de débugage...

Et quelque soit le mode utilisé rien n'apparait.

Le problème est apparu de mon coté après la dernière mise à jour (1.2.7)

J'ai tenté d'utiliser le RPC avec une autre application et ça marche donc il doit y avoir un souci avec la dernière MAJ :/

J'ai essayé de fournir le plus de détails possible en espérant que cela puisse aider !

BlocBlocTheDev commented 2 months ago

EDIT : Même en revenant sur la version précédente le problème est toujours présent.

JustYuuto commented 2 months ago

Ah sorry je vais voir ça quand j'ai le temps et puis je ferais une update, merci du retour en tous cas

BlocBlocTheDev commented 2 months ago

Pas de souci !

Ma piste c'est que Discord à changé un truc qui fait que l'application n'arrive plus à update le RPC. Mais bon vu que je connais rien à JS ou Discord je préfère pas trop m'avancer dessus.

Bon courage !

JuSteFat commented 2 months ago

La même, je pense que ça vient aussi de discord pour que ça arrête de marcher comme ça du jour au lendemain

flomirtech commented 2 months ago

Bonjour, je confirme également le problème sur la version actuelle et la précédente. Discord a dû changer une fonction lié au RPC avec Deezer 😅

TanukiPenny commented 2 months ago

Sorry to come in with English here lol, but I think Deezer changed something about their website that caused some of the mods that this program does to fail and break. That's why old versions do not work either cause they use the same website from Deezer. I just submitted PR #66 that should fix this!

JuSteFat commented 2 months ago

Sorry to come in with English here lol, but I think Deezer changed something about their website that caused some of the mods that this program does to fail and break. That's why old versions do not work either cause they use the same website from Deezer. I just submitted PR #66 that should fix this!

Can we test it ourselves by compiling the code?

TanukiPenny commented 2 months ago

Yes, if you pull my fork and build it you can test it

flomirtech commented 2 months ago

Yes, if you pull my fork and build it you can test it

How do you transform your fork into an .exe file ?

JuSteFat commented 2 months ago

Yes, if you pull my fork and build it you can test it

2024-08-18 20_05_19-Window Work guys gg you can use the pull

JuSteFat commented 2 months ago

Yes, if you pull my fork and build it you can test it

How do you transform your fork into an .exe file ?

I've managed to compile the application incorporating the pull request modifications. Here's an overview of the process:

  1. I applied the pull request changes manually in my local repository clone.

  2. I had to make several additional adjustments, notably in the package.json file and potentially in other configuration files.

  3. Next, I followed a compilation process similar to this one:

    • Installation of dependencies: npm install
    • TypeScript compilation: npm run build:ts
    • Executable creation: npm run build:electron
  4. The final executable is located in the dist folder.

It's important to note that the process required a few specific adaptations, not all of which are detailed here. If you'd like to reproduce exactly what I've done, I can provide more detailed instructions on the modifications made.

TanukiPenny commented 2 months ago

Pretty sure it's just 'yarn install' then 'yarn build'. Once the yarn build runs then the installer should be in a dist folder or a build folder. You should not have to modify anything except the changes I made in the pr

JuSteFat commented 2 months ago

Pretty sure it's just 'yarn install' then 'yarn build'. Once the yarn build runs then the installer should be in a dist folder or a build folder. You should not have to modify anything except the changes I made in the pr

Yes, but as I'd never compiled before, I followed claude-3.5's advice to use electron and make a portable version of deezer RPC. By the time it worked, I was happy.

BlocBlocTheDev commented 2 months ago

GG guys, now wait for a release in this repo :) 🍾🍾🍾

JuSteFat commented 2 months ago

GG guys, now wait for a release in this repo :) 🍾🍾🍾

I just make a fork with the pull if you want (using yarn install) : https://github.com/JuSteFat/deezer-discord-rpc/releases/tag/deezer

flomirtech commented 2 months ago

image Good Job @TanukiPenny & @JuSteFat. It works perfercly ! Thanks you very much 🍾

JustYuuto commented 1 month ago

Hi there, sorry for my inactivity here :(

Thank you @TanukiPenny and @JuSteFat for your help 😅, anyway I released another version that fixes some things :)

I'll leave this issue open in case anyone has the issue even with the latest update