Skip to content

Add telemetry docs #381

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 1 commit into
base: master
Choose a base branch
from
Open

Add telemetry docs #381

wants to merge 1 commit into from

Conversation

knolleary
Copy link
Member

Part of node-red/node-red#5102

Add docs for the new telemetry feature.

Verified

This commit was signed with the committer’s verified signature. The key has expired.
lilnasy Arsh
Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds documentation for the new telemetry feature as described in Node-RED issue #5102.

  • Added a new "Usage Telemetry" section in the User Guide with a link to the telemetry docs.
  • Created a new telemetry documentation page outlining what data is collected, why, and how users can opt out.

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File Description
docs/user-guide/index.md Added a new section and link for usage telemetry.
docs/telemetry/index.md Introduced a new page with comprehensive telemetry docs.

Comment on lines +70 to +71
You can change you made via the Editor Settings dialog at any time.

Copy link
Preview

Copilot AI Apr 29, 2025

Choose a reason for hiding this comment

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

The phrase 'You can change you made' appears to be a grammatical error. Consider revising it to something like 'You can change your preference via the Editor Settings dialog at any time.'

Suggested change
You can change you made via the Editor Settings dialog at any time.
You can change your preference via the Editor Settings dialog at any time.

Copilot uses AI. Check for mistakes.


With the usage telemetry feature enabled, in response to sending the information, Node-RED
is notified when there is a new version available. This will generate a notification
in the editor tp help you know when its time to upgrade.
Copy link
Preview

Copilot AI Apr 29, 2025

Choose a reason for hiding this comment

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

The word 'tp' seems to be a typo. It should likely be 'to', making the sentence read: 'in the editor to help you know when it's time to upgrade.'

Suggested change
in the editor tp help you know when its time to upgrade.
in the editor to help you know when it's time to upgrade.

Copilot uses AI. Check for mistakes.

@Steve-Mcl
Copy link
Contributor

Steve-Mcl commented May 1, 2025

Apart from the 2 typos copilot points out it reads well.

My personal take is we dont front load the information about benefits. e.g. how enabling telemetry permits update notifications, how it will help us make decisions on the direction of Node-RED project etc. I feel that could go very early in the document under the Why section.

An additional thought that occurred to me was since we are making this front and center and opt-in to enable, we could include a little more info - in particular installed modules and versions. That would provide another dimension making the data much richer. I appreciate though that you have likely considered this and have reasons for not including it here.

@Steve-Mcl Steve-Mcl removed their request for review May 1, 2025 08:18
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