Skip to content

Conversation

@mrubens
Copy link
Collaborator

@mrubens mrubens commented Sep 11, 2025

Important

Telemetry is now enabled by default, with UI and privacy policy updates to reflect this change, and users can opt-out via settings.

  • Behavior:
    • Telemetry is now enabled by default, users can opt-out via settings.
    • Updates updateTelemetryState() in TelemetryService.ts and webviewMessageHandler.ts to reflect default enabled state.
  • UI:
    • Simplifies TelemetryBanner in TelemetryBanner.tsx to remove allow/deny buttons, adds close button.
    • Displays TelemetryBanner only if telemetry setting is "unset".
  • Privacy Policy:
    • Updates PRIVACY.md to reflect telemetry is collected by default.
  • Translations:
    • Updates telemetry-related messages in multiple locale files to reflect default enabled state.
  • Misc:
    • Renames didUserOptIn to isOptedIn in TelemetryService.ts and telemetry.ts.

This description was created by Ellipsis for f86c630. You can customize this summary. It will automatically update as commits are pushed.

@mrubens mrubens requested review from cte and jr as code owners September 11, 2025 18:26
@dosubot dosubot bot added size:L This PR changes 100-499 lines, ignoring generated files. Enhancement New feature or request labels Sep 11, 2025
Copy link
Contributor

@roomote roomote bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for your contribution! I've reviewed the changes to make telemetry opt-out by default. I have some critical concerns about the privacy implications and UI behavior that need attention.

@hannesrudolph hannesrudolph added the Issue/PR - Triage New issue. Needs quick review to confirm validity and assign labels. label Sep 11, 2025
Co-authored-by: ellipsis-dev[bot] <65095814+ellipsis-dev[bot]@users.noreply.github.com>
@daniel-lxs daniel-lxs moved this from Triage to PR [Needs Review] in Roo Code Roadmap Sep 11, 2025
Copy link
Member

@daniel-lxs daniel-lxs left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me!

@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Sep 11, 2025
@hannesrudolph hannesrudolph added PR - Needs Review and removed Issue/PR - Triage New issue. Needs quick review to confirm validity and assign labels. labels Sep 11, 2025
@mrubens mrubens merged commit 33fe6fb into main Sep 11, 2025
14 of 15 checks passed
@mrubens mrubens deleted the default_telemetry branch September 11, 2025 19:50
@github-project-automation github-project-automation bot moved this from New to Done in Roo Code Roadmap Sep 11, 2025
@github-project-automation github-project-automation bot moved this from PR [Needs Review] to Done in Roo Code Roadmap Sep 11, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Enhancement New feature or request lgtm This PR has been approved by a maintainer PR - Needs Review size:L This PR changes 100-499 lines, ignoring generated files.

Projects

No open projects
Archived in project

Development

Successfully merging this pull request may close these issues.

3 participants