Skip to content

feat(web): always show the usage-data opt-out toggle in settings - #1232

Merged
wbxl2000 merged 1 commit into
mainfrom
feat/web-telemetry-toggle
Jun 30, 2026
Merged

feat(web): always show the usage-data opt-out toggle in settings#1232
wbxl2000 merged 1 commit into
mainfrom
feat/web-telemetry-toggle

Conversation

@wbxl2000

Copy link
Copy Markdown
Collaborator

Related Issue

No linked issue. This addresses a UX gap found while reviewing the web settings.

Problem

The telemetry opt-out toggle in the web settings was only rendered when the config explicitly contained a telemetry value, so it was hidden in the common default (unset) case. Its on/off mapping also treated the unset default as off, even though telemetry is enabled by default on the backend, which made the switch misleading and meant clicking it wrote a redundant true instead of actually opting out.

What changed

  • Always render the toggle in the web settings instead of gating it on the config value.
  • Treat unset / true as on and only explicit false as off, matching the backend's opt-out semantics, and fix the click handler so an unset state flips to false when toggled.
  • Rename it to "Improve product with usage data" / "使用数据改进产品", add a description of what is collected, and note that the change takes effect after restarting the service.

Checklist

  • I have read the CONTRIBUTING document.
  • I have linked a related issue, or explained the problem above.
  • I have added tests that prove my feature works. (UI visibility/copy change; vue-tsc typecheck passes.)
  • Ran gen-changesets skill, or this PR needs no changeset.
  • Ran gen-docs skill, or this PR needs no doc update.

The telemetry toggle was hidden unless the config explicitly set a value, and its on/off mapping treated the default (unset) state as off even though telemetry is enabled by default. Show it always, treat unset/true as on, and rename it with a description and a restart note.
@changeset-bot

changeset-bot Bot commented Jun 30, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 6fe0643

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@moonshot-ai/kimi-code Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@pkg-pr-new

pkg-pr-new Bot commented Jun 30, 2026

Copy link
Copy Markdown
pnpm dlx https://pkg.pr.new/@moonshot-ai/kimi-code@6fe0643
npx https://pkg.pr.new/@moonshot-ai/kimi-code@6fe0643

commit: 6fe0643

@wbxl2000
wbxl2000 merged commit aa6b0d0 into main Jun 30, 2026
10 checks passed
@wbxl2000
wbxl2000 deleted the feat/web-telemetry-toggle branch June 30, 2026 12:49
@github-actions github-actions Bot mentioned this pull request Jun 30, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant