search

SunResearchInstitute / SwitchPresence-Rewritten

Nintendo Switch sysmodule server for PresenceClient!
https://link.sunthecourier.net/switchpresence
GNU General Public License v2.0
126 stars 15 forks source link

GUI Client: Add an option to display Main Menu as status #27

Closed cathery closed 4 years ago

cathery commented 4 years ago

This is the compromise from my PRs in the past. I wish there was an option to not display main menu status on rich presence, because i'm often idle and it bugs the frick out of me.

ioistired commented 4 years ago

:+1: I would like this feature as well, but I'd like it in the CLI client. I wonder if it would be better to put this in the Server instead. If so configured, avoid sending updates for the main menu title at all. That way it would not need to be supported in both the GUI and the CLI client.

cathery commented 4 years ago

I would add this to the CLI client as well, but I didn't see how to pass any options besides the initial IP and ID, so I didn't do anything without further instruction.

gabehxd commented 4 years ago

I was thinking about adding https://github.com/commandlineparser/commandline to help with options but im still not sure if thats the best approach for options

ioistired commented 4 years ago

Argument parsing works. For a long-running process, it's more common to have a separate configuration file. Something like client --config /path/to/config would do nicely. Something more human-friendly than JSON (no comments, bothersome quoting rules for non-developers), such as INI or TOML would be preferred.