PC window: show the first-run setup once

CHANGELOG.md

@@ -1,5 +1,9 @@
 # Changelog
 
+## 2.2.3
+
+- **PC window setup shows once.** The first-run setup was only marked done by **Done** on its last step, so closing the window part-way (the main screen has the same QR codes) started it over on every open. It's now marked done as soon as it's shown.
+
 ## 2.2.2
 
 - **Phone app updates.** Tapping **Update** more than once (or on both the home screen and in Settings) started downloads that overwrote each other, and the update then failed with "The download is corrupt (checksum mismatch)". Only one download runs now, its button shows it's downloading, each download goes to a file of its own, and a damaged download is retried once. If your phone shows that error on the way to this version, tap **Update** once and wait, or install `PowerDeck.apk` from the release page.

helper/internal/localui/web/app.js

@@ -194,8 +194,6 @@ function showStep(n) {
 }
 
 function finishSetup() {
-  setting({ setupDone: true }).catch(() => {});
-  if (state) state.settings.setupDone = true;
   showStep(0);
   refresh();
 }
@@ -219,6 +217,8 @@ async function refresh() {
     started = true;
     link($("app-link"), s.appUrl);
     link($("app-link-home"), s.appUrl);
+    // Setup is shown once: closing the window part-way counts too, since home has the same codes.
+    if (!settings.setupDone) setting({ setupDone: true }).catch(() => {});
     showStep(settings.setupDone ? 0 : 1);
   }
 

helper/internal/store/store.go

@@ -134,7 +134,7 @@ type Settings struct {
 	NoAgents bool `json:"noAgents,omitempty"`
 	// Tray shows PowerDeck's icon in the Windows notification area.
 	Tray bool `json:"tray,omitempty"`
-	// SetupDone records that the PC window's first-run setup was finished or skipped.
+	// SetupDone records that the PC window has shown its first-run setup (once is enough).
 	SetupDone bool `json:"setupDone,omitempty"`
 	// UpdatesPausedUntil holds automatic updates back until then.
 	UpdatesPausedUntil time.Time `json:"updatesPausedUntil,omitzero"`