Skip to content

feat(widgets): Saved filters in widgets: Remove FF (#15689) - #17523

Merged
Archidoit merged 1 commit into
masterfrom
issue/15689-ff
Aug 5, 2026
Merged

feat(widgets): Saved filters in widgets: Remove FF (#15689)#17523
Archidoit merged 1 commit into
masterfrom
issue/15689-ff

Conversation

@Archidoit

Copy link
Copy Markdown
Member

Proposed changes

Remove the feature flag DASHBOARD_SAVED_FILTERS for the Saved filters in widget feature

Related issues

fixes #15689

Copilot AI lite review requested due to automatic review settings August 5, 2026 10:00
@Archidoit Archidoit self-assigned this Aug 5, 2026
@Archidoit Archidoit added the filigran team Item from the Filigran team. label Aug 5, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR removes the DASHBOARD_SAVED_FILTERS feature flag gating for the “saved filters in widgets” capability in the OpenCTI frontend, making the feature available by default where widgets support it.

Changes:

  • Removed useHelper().isFeatureEnable('DASHBOARD_SAVED_FILTERS') gating from WidgetFilters and now rely only on widget host kind for access control.
  • Updated unit tests by removing the useHelper mock that previously forced the feature flag to be enabled.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
opencti-platform/opencti-front/src/private/components/widgets/WidgetFilters.tsx Removes feature-flag check and keeps saved-filters accessibility based on host.kind only.
opencti-platform/opencti-front/src/private/components/widgets/WidgetFilters.test.tsx Removes the feature-flag mock to align tests with the new always-on behavior.

@codecov

codecov Bot commented Aug 5, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 34.11%. Comparing base (f73357b) to head (462bc3d).

Additional details and impacted files
@@           Coverage Diff           @@
##           master   #17523   +/-   ##
=======================================
  Coverage   34.11%   34.11%           
=======================================
  Files        3376     3376           
  Lines      137662   137660    -2     
  Branches    37258    37258           
=======================================
  Hits        46963    46963           
+ Misses      90699    90697    -2     
Flag Coverage Δ
opencti-client-python 48.37% <ø> (ø)
opencti-front 11.07% <100.00%> (-0.01%) ⬇️
opencti-graphql 69.12% <ø> (+<0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@Archidoit
Archidoit merged commit d2e1ac9 into master Aug 5, 2026
59 checks passed
@Archidoit
Archidoit deleted the issue/15689-ff branch August 5, 2026 15:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

filigran team Item from the Filigran team.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

feat(dashboards): reusable saved filters in dashboards

3 participants