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

Roaming folder deleted? #769

Closed
atypicalprogrammer opened this issue Sep 22, 2016 · 3 comments · May be fixed by qcif/data-curator#563
Closed

Roaming folder deleted? #769

atypicalprogrammer opened this issue Sep 22, 2016 · 3 comments · May be fixed by qcif/data-curator#563

Comments

@atypicalprogrammer
Copy link

  • Version: latest
  • Target: NSIS

Why is the roaming folder deleted during install? (AppData/Roaming/App Name/) and how can I prevent this? This contains files that should never be deleted on install / upgrade.

@develar
Copy link
Member

develar commented Sep 23, 2016

We do not remove app data on auto uninstall (as part of installing new version) — upgrade, but we remove on uninstall.

Please specify do you mean that app data must be not removed on uninstall (Add or Remove Programms -> Uninstall)?

@develar develar added the nsis label Sep 23, 2016
@ghost
Copy link

ghost commented Sep 23, 2016

Please specify do you mean that app data must be not removed on uninstall (Add or Remove Programms -> Uninstall)?

Correct. This should not happen without explicit user consent.

@atypicalprogrammer
Copy link
Author

atypicalprogrammer commented Sep 23, 2016

Well, for me data is removed on auto uninstall as part of installing new version. So I don't know why that is. It happens during the NSIS installation phase.

(But the ability to allow the user to choose to keep data on complete uninstall would be welcomed too). It would be nice if the app would receive a notification and could remove and keep what it wants to. But I digress, my main issue is on auto uninstall data gets removed that shouldn't.

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

Successfully merging a pull request may close this issue.

2 participants