-
Notifications
You must be signed in to change notification settings - Fork 59
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
Ensure telemetry is enabled for licensed installs #1736
Conversation
@@ -26,6 +27,7 @@ module.exports = async function (app) { | |||
} | |||
|
|||
if (request.session.User.admin) { | |||
response['platform:licensed'] = isLicensed |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not 100% sure about including this here as we already have the /api/v1/admin/license
end point that returns license details.
That said, we can move forward with this as-is and iterate if needed.
@@ -22,6 +22,8 @@ Settings section of the platform UI. | |||
It is also possible to disable in the `flowforge.yml` configuration file. This | |||
overrides whatever option is set in the Admin Settings UI. | |||
|
|||
**IMPORTANT: Licensed installations cannot disable telemetry** |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good first iteration, but I think we need to word this slightly differently as we don't talk about 'installations' generally.
We will need to document the behaviours around licensing in general with the changes coming in the other tasks. So lets make sure we do a 'big picture' review of the licensing docs as part of this work once we have the full set of behaviours defined.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Description
Outdated (click to reveal)
pre 2023/02/22
telemetry:enabled
setting is always true if product is licensedplatform:licensed
boolean setting for the following 2 items...Updated 2023/02/22
telemetry:enabled
setting is always true if product is licensed (both client and server side)platform:licensed
boolean (readonly) settingplatform:licensed
setting to hide telemetry setting checkbox and parent group titleRelated Issue(s)
#1640
Checklist
flowforge.yml
?flowforge/helm
to update ConfigMap Templateflowforge/CloudProject
to update values for Staging/ProductionLabels
backport
labelarea:migration
label