Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Set telemetry to disabled by default #4452

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

jonvargason
Copy link

@jonvargason jonvargason commented Mar 3, 2025

Description

Disabling telemetry by default. Why: Some users and corporations may not want to have an opt-out model but would rather have an opt-in to avoid accidentally sending potentially company sensitive data by accident upon install. Full disclosure I don't believe anything this plugin sends is sensitive but companies are very concerned about data egress. I personally think we should enable it for academic and personal usage but enterprise adoption - this may be frowned on or likely would be blocked anyway by firewalls.

Checklist

  • The relevant docs, if any, have been updated or created
  • [] The relevant tests, if any, have been updated or created

Screenshots

image

image

After install:
image

Testing instructions

I'll be honest and say I could not get this to build on my work network but I suspect it was a proxy thing and VS issue as even though I made what I believe are the correct modifications and pointed to my instance of VS 2022 it still failed on sqlite and I am not sure why. I was able to pull the pipeline build at https://github.com/continuedev/continue/actions/runs/13640223539/job/38128264263?pr=4452.

I believe to test this you basically follow the instructions on contributing.

  1. Make sure you have the correct version of npm (v20.11.0)
  2. Pull down my PR
  3. Install all dependencies in VS Code
  4. Build and Run Extension locally
  5. Validate Telemetry is disabled by default (settings -> unchecked Telemetry Enabled)

Alternatively if you want to skip the build like I did locally:

  1. Grab that binary at https://github.com/continuedev/continue/actions/runs/13640223539/job/38128264263?pr=4452.
  2. Then Open VS Code and Install VSIX
  3. Validate Telemetry is disabled by default (settings -> unchecked Telemetry Enabled)

Copy link

netlify bot commented Mar 3, 2025

Deploy Preview for continuedev ready!

Name Link
🔨 Latest commit 9757143
🔍 Latest deploy log https://app.netlify.com/sites/continuedev/deploys/67c628643ebf190008571e13
😎 Deploy Preview https://deploy-preview-4452--continuedev.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

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