digimezzo / dopamine

The audio player that keeps it simple
https://digimezzo.github.io/site/
GNU General Public License v3.0
1.49k stars 111 forks source link

Uninstalling dopamine should delete the AppData/Roaming/Dopamine and AppData/Local/dopamine-updater folders #534

Closed DopamineUser closed 9 months ago

DopamineUser commented 9 months ago

and for some strange reason, i have to restart my pc after deleting all dopamine files before installing it again in order to have it start fresh, else it loads everything back somehow???

digimezzo commented 9 months ago

That's a difficult one. Deleting AppData/Roaming/Dopamine means losing all downloaded album art, settings, Dopamine.db (including ratings). So that'll probably not happen soon. Having to restart the computer in order to start fresh is a strange and unexpected one. I'll do some tests in Windows.

DopamineUser commented 9 months ago

I've seen uninstallers with a checkbox to erase configurations or everything, that would be great.

digimezzo commented 9 months ago

@DopamineUser That should be possible. I'm adding this link to assist myself during development: https://medium.com/@mohitkaushal91/how-to-build-package-and-distribute-your-cross-platform-desktop-apps-electron-builder-935ed44d40d1

I'll try to provide that in preview 23, which I'll release in a few days.

Other NSIS options: https://www.electron.build/configuration/nsis.html

deleteAppDataOnUninstall does not seem to be optional for the user (it's either true of false). This script might be a solution: https://github.com/electron-userland/electron-builder/issues/4141

Sidebar.zip