Enable Productivity Analytics feature
What does this MR do?
Sets Productivity Analytics feature flags to default to "on".
This was requested today by @valexieva in Slack#g_manage_analytics:
ahegyi 7 hours ago what are the two flags?
virjinia 5 hours ago productivity_analytics
virjinia 5 hours ago productivity_analytics_scatterplot_enabled (not sure why it says enabled)
ahegyi 2 hours ago did you turn it on or off? I can see the scatterplot
virjinia 2 hours ago it's on, but I want to make it default on for on premise. I believe if the code says default on, then it ships as on in on-premise
virjinia 2 hours ago isn't that the case?
ahegyi 2 hours ago productivity_analytics_scatterplot_enabled, default_enabled: true
virjinia 2 hours ago cool, can we make the same for productivity_analytics?
virjinia 2 hours ago otherwise, unsure what actually happens?
ahegyi 2 hours ago if the 'productivity_analytics' ff is off then PA won't be visible at all (including scatterplot)
virjinia 2 hours ago
ok, can we make it default on, please?
ahegyi 2 hours ago sure, I'll submit an MR
virjinia 2 hours ago Thanks!
Screenshots
Does this MR meet the acceptance criteria?
Conformity
-
Changelog entry -
Documentation created/updated or follow-up review issue created -
Code review guidelines -
Merge request performance guidelines -
Style guides -
Database guides -
Separation of EE specific content
Availability and Testing
-
Review and add/update tests for this feature/bug. Consider all test levels. See the Test Planning Process. -
Tested in all supported browsers
Security
If this MR contains changes to processing or storing of credentials or tokens, authorization and authentication methods and other items described in the security review guidelines:
-
Label as security and @ mention @gitlab-com/gl-security/appsec
-
The MR includes necessary changes to maintain consistency between UI, API, email, or other methods -
Security reports checked/validated by a reviewer from the AppSec team
EDITED same day by @djensen to add Slack transcript.