Skip to content

chore: ga helios insights [ENG-3734]#8130

Open
speaker-ender wants to merge 1 commit intomainfrom
chore/ga-helios-insights
Open

chore: ga helios insights [ENG-3734]#8130
speaker-ender wants to merge 1 commit intomainfrom
chore/ga-helios-insights

Conversation

@speaker-ender
Copy link
Copy Markdown
Contributor

@speaker-ender speaker-ender commented May 7, 2026 •
edited
Loading

Ticket ENG-3734

Description Of Changes

Removing HeliosInsights flag

Code Changes

Steps to Confirm

  1. Confirm that the helios insights flag no longer exists in the admin settings
  2. Verify that the statistics widget always displays in the action center

Pre-Merge Checklist

  • Issue requirements met
  • All CI pipelines succeeded
  • CHANGELOG.md updated
    • Add a db-migration This indicates that a change includes a database migration label to the entry if your change includes a DB migration
    • Add a high-risk This issue suggests changes that have a high-probability of breaking existing code label to the entry if your change includes a high-risk change (i.e. potential for performance impact or unexpected regression) that should be flagged
    • Updates unreleased work already in Changelog, no new entry necessary
  • UX feedback:
    • All UX related changes have been reviewed by a designer
    • No UX review needed
  • Followup issues:
    • Followup issues created
    • No followup issues
  • Database migrations:
    • Ensure that your downrev is up to date with the latest revision on main
    • Ensure that your downgrade() migration is correct and works
      • If a downgrade migration is not possible for this change, please call this out in the PR description!
    • No migrations
  • Documentation:
    • Documentation complete, PR opened in fidesdocs
    • Documentation issue created in fidesdocs
    • If there are any new client scopes created as part of the pull request, remember to update public-facing documentation that references our scope registry
    • No documentation updates required

@vercel
Copy link
Copy Markdown
Contributor

vercel Bot commented May 7, 2026 •
edited
Loading

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
fides-plus-nightly Ready Ready Preview, Comment May 7, 2026 1:11pm
1 Skipped Deployment
Project Deployment Actions Updated (UTC)
fides-privacy-center Ignored Ignored May 7, 2026 1:11pm

Request Review

@speaker-ender
chore: ga helios insights [ENG-3734]
1066d44 
chore: update changelog
@speaker-ender speaker-ender force-pushed the chore/ga-helios-insights branch from 3567fed to 1066d44 Compare May 7, 2026 13:09
@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 7, 2026

Title Lines Statements Branches Functions
admin-ui Coverage: 8%
 6.6% (3042/46031) 5.92% (1572/26513) 4.62% (630/13611)
fides-js Coverage: 78%
 79.56% (2028/2549) 66.4% (1259/1896) 73.31% (349/476)
privacy-center Coverage: 85%
 82.53% (364/441) 79.74% (189/237) 74.07% (60/81)

@speaker-ender speaker-ender marked this pull request as ready for review May 7, 2026 13:46
@speaker-ender speaker-ender requested a review from a team as a code owner May 7, 2026 13:46
@speaker-ender speaker-ender requested review from gilluminate and removed request for a team May 7, 2026 13:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@gilluminate gilluminate gilluminate approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@speaker-ender @gilluminate