Skip to content

docs: Copilot privacy concerns #4325

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

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

Conversation

MarcinVaadin
Copy link
Member

Added Privacy Concerns section with more detailed information about what is being sent and how to opt out.

@MarcinVaadin MarcinVaadin requested a review from emarc May 22, 2025 09:02
Copy link

github-actions bot commented May 22, 2025

AI Language Review

The review reveals that the main addition to the document is the "Privacy" section, outlining how Copilot handles data sharing and privacy. Here are a few suggested improvements for this new section:

  1. Clarity:

    • In the "Shared data" column, instead of "Source code related to given view, depending on operation," consider specifying what "given view" refers to for more clarity.
    • In "Copilot error" row, specify what constitutes "Telemetry" and ensure it's clear what data is included in "Vaadin dependencies versions" to avoid ambiguity.
  2. Consistency:

    • The opt-out instructions should consistently guide users to the Vaadin settings or system properties sections. If one method is preferable, highlight it.
  3. Formatting:

    • Ensure that system properties (vaadin.devmode.usageStatistics.enabled) are formatted consistently throughout the text and with inline code syntax for easier reading.

These changes should improve the clarity and consistency of the new section's content.

@emarc
Copy link
Contributor

emarc commented Jun 23, 2025

Looks goo, I suggest the following adjustments:

  1. Name the section just "Privacy" (drop "concerns")
  2. "User is asked before first operation if AI usage is allowed." -> "User asked for permission before operation. This can be changed via Settings > AI Usage"
  3. "Command Window prompt submitted": add "User asked for permission before operation (same as above.)"
  4. "User Activity" -> "Copilot usage" (not super good, but works well with the next one)
  5. "Copilot error happened" -> "Copilot error"

@emarc
Copy link
Contributor

emarc commented Jun 23, 2025

"Command Window prompt submitted": add "User asked for permission before operation (same as above.)"

Now the part about settings about settings on vaadin.com is completely dropped?

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.

2 participants