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

Save path of output directory of last exported files #6

Closed
manuel-mauky opened this issue Aug 27, 2019 · 4 comments · Fixed by #9
Closed

Save path of output directory of last exported files #6

manuel-mauky opened this issue Aug 27, 2019 · 4 comments · Fixed by #9

Comments

@manuel-mauky
Copy link
Contributor

First of all: Thanks for this app. It helps me a lot.

Would be great to save the selected path when a file is exported so that the next time you don't have to select the path again and again.

I'm happy to send a PR for this.

@vertongen
Copy link
Owner

great suggestion! If you have time to make a PR, feel free to do so and I'll include it in a release.

@vertongen
Copy link
Owner

One thought: where would you store that 'preference' ?
Right now the keychain is the only 'storage' the program uses, which is dedicated to the credentials of the user. So it would have to be another place?

@manuel-mauky
Copy link
Contributor Author

Good point. I've just thought "save it the same way as the credentials" but you are correct. The keychain isn't the correct place to do it.

I don't know what the correct solution would be. In linux I know that many programs create a .programm-name folder in the users home and store their configs there. On mac-os some programs also do this but I don't know if this is the real way of doing it. In windows you might use also use the registry.

Is there some abstraction/API from electron for storing preferences?

@vertongen
Copy link
Owner

My suggestion would be to use a library to store data, something like this: https://github.com/electron-userland/electron-json-storage
Or it can be another one if you have other suggestions.

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 a pull request may close this issue.

2 participants