Skip to content

Feat(Analytics/Embedded Insights): add new page

Michael Lunøe requested to merge 224191-add-embedded-insights-sibling-page into master

What does this MR do?

This MR adds a new (blank) sibling page to the Analytics Insights page that can be embedded. This will be used to display a single chart from the Insights page that can be embedded into other pages.

This page is behind embed_analytics_report feature flag added here and only works for public groups/projects:

Screenshots

Blank screen for now. Does not contain anything yet:

Does this MR meet the acceptance criteria?

Conformity

Availability and Testing

To test this out run bin/rake rails c and run the following to activate the feature flag: Feature.enable(:embed_analytics_report, Project.find_by_full_path("Commit451/lab-coat")) Then navigate to http://127.0.0.1:3000/Commit451/lab-coat/insights/embedded to see the page.

Also, to try to embed this into, say the insights page, apply this patch and navigate to http://127.0.0.1:3000/Commit451/lab-coat/insights: test_embedded.patch

Minimal risk since this is behind the feature flag.

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

Relates to #224191 (closed)

Edited by Michael Lunøe

Merge request reports