jkarasti

One note to add here: The windows installer could also be localised. https://wixtoolset.org/docs/tools/wixext/wixui/#localization

> I tested with `Scope="perUser"` and I get the following error: > >  > > This is similar to this [SO post](https://stackoverflow.com/questions/77839235/wix-how-to-upgrade-per-user-installation-including-services), in which they had to do some...

> ## Should `AllowSameVersionUpgrades="yes"` be removed? > > My understanding is that previously, we specified `*` as the product code: > > https://github.com/freedomofpress/dangerzone/blob/791444cd5dd983f87696c4d16313f2d3a6af515d/install/windows/build-wxs.py#L167 > > With this PR, we don't...

Pushed a couple of changes: #### Removed the `InstallerVersion`. Let's trust the WiX developers keep the default at a sensibly old version of Windows. #### Updated the build instructions. Better...

> Hey @jkarasti. Just wanted to let you know that we have wrapped Dangerzone 0.8.0. If there's something I can take a look at or help you with, now is...

> Just checked the diff between the state of the PR when I reviewed it and its current state. I'm fine with the changes, and the only thing remaining is...

Trying to avoid leaving the Scope issue as is, I took an another deep dive to WiX Toolset `Launch` and `Custom Action` elements. Unfortunately there really doesen't seem to be...

> Just tested [jkarasti:602-upgrade-wix-toolset-fix-regsearch](https://github.com/jkarasti/dangerzone/tree/602-upgrade-wix-toolset-fix-regsearch) and it detected the Dangerzone 0.8.0 installation. However, it doesn't detect previous installations, such as 0.6.0. Our users may skip 0.8.0 for whatever reasons, so we...