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

Allow users to say no when writing to fishnet.ini #263

Merged

Conversation

nobriot
Copy link
Contributor

@nobriot nobriot commented Mar 29, 2024

Got myself in an irritating loop where the user has to say yes.
I understand that writing the file is necessary to provide a service file, but I ended up doing Ctrl+C instead of saying yes.

Got myself in an irritating loop where the user has to say yes.
I understand that writing the file is necessary to provide a service
file, but I ended up doing Ctrl+C instead of saying yes.
@niklasf niklasf merged commit 8c5e24c into lichess-org:master Mar 30, 2024
6 checks passed
@niklasf
Copy link
Member

niklasf commented Mar 30, 2024

Looks good. Thanks!

@nobriot
Copy link
Contributor Author

nobriot commented Mar 30, 2024

Actually it could also be quite nice if it would ask the user where to save the file. (I like it at $HOME/.config/fishnet/fishnet.ini for linux).

The first time it just says fishnet.ini, which kind of saves it in a random place depending on where you ran the executable as far as I can see... (e.g. $HOME/Downloads).

@niklasf
Copy link
Member

niklasf commented Apr 1, 2024

If we can keep compatibility for existing installations, generally using the paths designated by the operating system seems like a good idea.

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

Successfully merging this pull request may close these issues.

2 participants