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).
This commit is contained in:
Louis Knight-Webb
2025-07-12 06:54:04 +01:00
committed by GitHub
parent b7230006c6
commit 8a8c7a16f6
2 changed files with 29 additions and 4 deletions

View File

@@ -163,7 +163,7 @@ export function OnboardingDialog({ open, onComplete }: OnboardingDialogProps) {
disabled={!isValid}
className="w-full"
>
Complete Setup
Continue
</Button>
</DialogFooter>
</DialogContent>

View File

@@ -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>