From 8a8c7a16f63375bb0d1ad6a3467ebbb886574057 Mon Sep 17 00:00:00 2001
From: Louis Knight-Webb <louis@bloop.ai>
Date: Sat, 12 Jul 2025 06:54:04 +0100
Subject: [PATCH] Update settings (vibe-kanban) (#148)

* Added support for `telemetry_acknowledged` in the Privacy section. It follows the same pattern as the disclaimer and onboarding acknowledgments with:

- Status display showing whether it's been acknowledged
- Reset button to clear the acknowledgment (disabled when not acknowledged)
- Explanatory text about what happens when reset
- Visual separation from the analytics toggle with a border

The field will now be properly managed in the Settings UI and can be reset if needed, requiring re-acknowledgment on the next app start.

* Moved the telemetry acknowledgment from the Privacy section to the Safety & Disclaimers section. It's now grouped with the other acknowledgments (disclaimer and onboarding) which makes more logical sense since they all follow the same pattern of managing user acknowledgments that can be reset to show notices again on app startup.

* Updated all three disclaimer texts to remove "on next app start" since users will need to acknowledge them immediately when reset, not on the next app startup.

* Changed the button text from "Complete Setup" to "Continue" in the OnboardingDialog component to better reflect that there are additional onboarding steps after this initial setup.

* Prettier formatting completed successfully! All files have been formatted, with only the Settings.tsx file having been updated (likely due to our recent changes).
---
 frontend/src/components/OnboardingDialog.tsx |  2 +-
 frontend/src/pages/Settings.tsx              | 31 ++++++++++++++++++--
 2 files changed, 29 insertions(+), 4 deletions(-)

diff --git a/frontend/src/components/OnboardingDialog.tsx b/frontend/src/components/OnboardingDialog.tsx
index a38e3559..3151366c 100644
--- a/frontend/src/components/OnboardingDialog.tsx
+++ b/frontend/src/components/OnboardingDialog.tsx
@@ -163,7 +163,7 @@ export function OnboardingDialog({ open, onComplete }: OnboardingDialogProps) {
             disabled={!isValid}
             className="w-full"
           >
-            Complete Setup
+            Continue
           </Button>
         </DialogFooter>
       </DialogContent>
diff --git a/frontend/src/pages/Settings.tsx b/frontend/src/pages/Settings.tsx
index 7493cc1d..cb3eac1b 100644
--- a/frontend/src/pages/Settings.tsx
+++ b/frontend/src/pages/Settings.tsx
@@ -517,7 +517,7 @@ export function Settings() {
                 </div>
                 <p className="text-xs text-muted-foreground">
                   Resetting the disclaimer will require you to acknowledge the
-                  safety warning again on next app start.
+                  safety warning again.
                 </p>
               </div>
               <div className="space-y-2">
@@ -540,8 +540,33 @@ export function Settings() {
                   </Button>
                 </div>
                 <p className="text-xs text-muted-foreground">
-                  Resetting the onboarding will show the setup screen again on
-                  next app start.
+                  Resetting the onboarding will show the setup screen again.
+                </p>
+              </div>
+              <div className="space-y-2">
+                <div className="flex items-center justify-between">
+                  <div>
+                    <Label>Telemetry Acknowledgment</Label>
+                    <p className="text-sm text-muted-foreground">
+                      {config.telemetry_acknowledged
+                        ? 'You have acknowledged the telemetry notice.'
+                        : 'The telemetry notice has not been acknowledged.'}
+                    </p>
+                  </div>
+                  <Button
+                    onClick={() =>
+                      updateConfig({ telemetry_acknowledged: false })
+                    }
+                    variant="outline"
+                    size="sm"
+                    disabled={!config.telemetry_acknowledged}
+                  >
+                    Reset Acknowledgment
+                  </Button>
+                </div>
+                <p className="text-xs text-muted-foreground">
+                  Resetting the acknowledgment will require you to acknowledge
+                  the telemetry notice again.
                 </p>
               </div>
             </CardContent>