Open prettyv opened 1 year ago
This recommendation (relying on isTelemetryEnabled ) is fairly new, as I checked it a few weeks ago. We can certainly adapt to the new recommendation from now on.
Ah, the dedicated telemetry extension guide page was indeed just authored at the start of May and the wording adjusted to put more emphasis on respecting the global flag. Good to hear that there is willingness to adapt to this, much appreciated!
Another aspect I just noticed while going through the changes for the new version of vscode-xml
(Add usesOnlineServices tag) is that depending on which installed extension that provides the redhat.telemetry.enabled
setting is activated first that extension's setting properties win.
In this case not all Red Hat extensions I have installed locally specify the newly added tag for the setting so I didn't find it when filtering my settings by @tag:usesOnlineServices
. Concretely this time "Openshift.io configuration" was first, where I'm not even sure which extension this is coming from since I don't have any of the OpenShift extensions installed.
I would like to second this request. We see a popup message asking the user consent for feedback very often. This shouldn't appear and instead be using the global vscode telemetry setting.
Currently telemetry settings are exclusively controlled via the custom
redhat.telemetry.enabled
setting, which needs to be specified in each Red Hat extension'spackage.json
as a configuration property, leading to warning messages in the Developer Console when multiple Red Hat extensions are installed, as e.g. reported here:The official Telemetry extension authors guide mentions in Without the telemetry module:
and then in Custom telemetry settings:
The mentioned
isTelemetryEnabled
boolean setting is part of the VS Code API'sdev
namespace: https://code.visualstudio.com/api/references/vscode-api#envWould it be feasible to eventually migrate over to the global setting or are there other concerns around this?