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

Move config file to Roaming Appdata folder #17

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

feyyd
Copy link

@feyyd feyyd commented Jun 9, 2023

Moves the config files location to:

C:\Users<user>\AppData\Roaming\EVE-O Preview\EVE-O Preview.json

This should allow for installs from anywhere.

@iShaymus
Copy link

iShaymus commented Jul 4, 2023

Why? The most non technical users having EVE-O run like a portable app with all the files in a single folder is an advantage.

Not putting the idea down, just trying to understand the advantage. If it's to have the settings persist then a change should also be made to the client to reset all configs so that basic users don't have to try and find the config file to delete it

@feyyd
Copy link
Author

feyyd commented Jul 7, 2023

It was a feature request in the github page, I merely implemented it because it was requested. What happens to it after that isn't really my problem.

@AnrDaemon
Copy link

Why?

Because.
As an example, EVEMon also could be run in portable mode. Or in standard mode. It just checks if a config file exists in the same directory as .exe, and works up from that.

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.

3 participants