Skip to content

feat: Add optional force option to updateRemoteFeatureFlags - #9966

Merged
Cal-L merged 2 commits into
mainfrom
feat/add-force-option-for-fetching-flags
Aug 26, 2026
Merged

feat: Add optional force option to updateRemoteFeatureFlags#9966
Cal-L merged 2 commits into
mainfrom
feat/add-force-option-for-fetching-flags

Conversation

@Cal-L

@Cal-L Cal-L commented Aug 25, 2026

Copy link
Copy Markdown
Contributor

Explanation

This PR just adds an optional force arg to updateRemoteFeatureFlags for forcing an update whenever it's set to true.
Example use case: We'll want to force an update once canonical IDs are available on the platform sides so that the proper flag variants are served

References

Checklist

  • I've updated the test suite for new or updated code as appropriate
  • I've updated documentation (JSDoc, Markdown, etc.) for new or updated code as appropriate
  • I've communicated my changes to consumers by updating changelogs for packages I've changed
  • I've introduced breaking changes in this PR and have prepared draft pull requests for clients and consumer packages to resolve them

Note

Low Risk
Backward-compatible optional parameter with narrow cache-bypass behavior; existing callers unchanged when omitting force.

Overview
Adds an optional force argument (default false) to updateRemoteFeatureFlags, so callers can refresh remote flags from the API even when the cache is still valid—e.g. after a canonical profile ID becomes available so threshold segmentation can be re-fetched.

When force is true, the early return that skipped fetch on a fresh cache is bypassed; disabled controllers still never fetch, regardless of force. Messenger action types, JSDoc, changelog, and tests cover the enabled + forced path and the disabled + forced no-op.

Reviewed by Cursor Bugbot for commit 1b71805. Bugbot is set up for automated code reviews on this repo. Configure here.

@Cal-L
Cal-L requested review from a team as code owners August 25, 2026 21:54
@Cal-L
Cal-L deployed to default-branch August 25, 2026 21:54 — with GitHub Actions Active
@DDDDDanica

Copy link
Copy Markdown
Contributor

LGTM !

@Cal-L
Cal-L added this pull request to the merge queue Aug 26, 2026
Merged via the queue into main with commit f1ceb4b Aug 26, 2026
378 of 401 checks passed
@Cal-L
Cal-L deleted the feat/add-force-option-for-fetching-flags branch August 26, 2026 18:32
@cursor cursor Bot mentioned this pull request Aug 26, 2026
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants