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

Change product name to fix Discord status conflict #131

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

ktwrd
Copy link

@ktwrd ktwrd commented Mar 14, 2023

Please make sure to check the following tasks before opening and submitting a PR

  • I understand and have followed the contribution guide
  • I have tested my changes locally and they are working as intended
  • These changes do not have any notable side effects on other Revolt projects

note: This PR replaces my previous one, #121

Changed property productName and build.appx.displayName in package.json to fix Discord Status, which then changes the Windows Executable to Revolt Desktop.exe instead of Revolt.exe to not get confused with the game Re-Volt.

Tested and working;

  • Windows Installer
  • Windows Portable (only difference is that the filename is changed, duh)
  • AppX Windows (Unaffected)
    • note: display name will change from Revolt to Revolt Desktop
  • macOS
    • unable to test myself since I do not have anything running macOS
  • Linux (AppImage)
    • AppImage will start with Revolt Desktop- instead of Revolt-.
    • This will not break any pipelines since they use wildcards to move to the directory for publishing.
  • Linux (Archive)
    • Unaffected since it uses package.name instead of package.productName

@X-lem
Copy link

X-lem commented Jun 19, 2023

I personally would vote for this change. As other's have said this is a Discord issue, but there's basically a 0% chance that Discord will fix this. This is a simple change Revolt can make to bypass this Discord bug. I personally don't keep Revolt open simply because I don't want this faulty Discord status. I would assume other's do the same. Some people are going to slowly transition from Discord to Revolt and Imo this is a simple cleanup solution.

It seems others have had success changing the .exe file name, but I don't think this is possible for everyone. I'm on Windows 10 and installed Revolt through the app store. The app is installed in C:\Program Files\WindowsApps\... and even as administrator I have not found a way to update the exe file (permission denied).

@ktwrd
Copy link
Author

ktwrd commented Jun 22, 2023

it's been over 3mo since this PR opened and idfk why it hasn't been merged yet /shrug

@theAkito
Copy link

theAkito commented Jun 22, 2023

it's been over 3mo since this PR opened and idfk why it hasn't been merged yet /shrug

You asked for it, so here are a couple of potential answers.

  1. I know why you would have Discord & Revolt open at the same time, however you still have the choice to only have one of them open, preferably the open source solution.
  2. You could also manually change the displayed game name.
  3. I could not find information on the topic, whether Discord only checks the plain exe name, when setting status, but when it literally just does that, you can change the name of that file on your computer, rather than ruining it for everyone.
  4. As far as I know, the people working here are not involved with the company behind Discord. If they aren't paid by them to do shit like this, they shouldn't.
  5. It's most of the time bad practice to fix other projects' issues. In this case, it's supposed to be never fixed by the other project, but there are other reasons why it's still a bad idea, which follow in this list.
  6. You want to change the name of this huge program, due to a simple game nobody gives a damn about & probably nobody knows. It has 800+ reviews on Steam & costs a couple of bucks without any discount.
  7. Changing the binary name in the way proposed basically means entirely destroying the whole logic of it.
    i. The proposed name is much longer, than the previous one.
    ii. The proposed name has a space in it. That's just mind-boggling & makes your head shake until it falls off.
    iii. The proposed name adds an entire, comparatively long, word to the binary, which does not make any sense. Yeah, no kidding, I'm running an exe file on the desktop. Thanks for telling me the obvious. Like, what the heck.

So, instead of trying to **** the shit out of this binary & leave it to rot in some hidden alley at the back of an Alabama building, the name should stay the same, or the newly proposed name should keep the elegance by for example shortening it to rev.exe or something, rather than completely butchering it & hanging it on the wall as the ugliest painting in the world.

Shit on Discord btw. Stop using it, altogether!

@ktwrd
Copy link
Author

ktwrd commented Jul 28, 2023

You asked for it, so here are a couple of potential answers.
...

There is no need for this amount of toxicity. Actually, any toxicity. @theAkito

So I'll answer your comment in a manner that somewhat matches you.

  1. Not everyone has moved to Revolt and currently it isn't feature complete compared to Discord. It makes logical sense to have both open.
  2. From my amount of experience with Revolt. They want users to have a seamless experience and having custom rich presence is one factor so they can easily tell others what platform they've moved to. Asking the user to rename an executable (on windows) means you also have to change the shortcut, which some users may not know how to.
  3. Discord does identify "official" games by the executable/process file name.
  4. See answer no.2
  5. See answer no.2. Also we can't do anything about it other than changing the executable name.
  6. If something you're doing to the revolt client that depends on the executable name manually then you're doing something wrong. I made this PR to avoid issues. Afik everything in the desktop repo actually pulls from the package.json instead of hard-coding it.
  7. Ok, and?
    i. A PR was merged about 3 weeks ago which names it (on Linux) to Revolt Desktop.
    ii. Adding a space will not break other platforms such a Linux because it will use a - instead. Adding spaces to executable names are fine (especially on windows) since the file path to anything in your home folder usually contains a space because the account creation assumes your display name is the same as your username.

@ktwrd
Copy link
Author

ktwrd commented Jul 28, 2023

I've checked AppImage's and they are unaffected by pipelines.

  • AppImage will start with Revolt Desktop- instead of Revolt-.
  • This will not break any pipelines since they use wildcards to move to the directory for publishing.

@ktwrd
Copy link
Author

ktwrd commented Jul 28, 2023

Latest commit also just does a rebase from revoltchat/desktop/orign/master

@ktwrd
Copy link
Author

ktwrd commented Sep 27, 2023

bumping

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: 💬 Awaiting review
Development

Successfully merging this pull request may close these issues.

None yet

4 participants