feat(flags): gate Bridge top-up/KYC queries behind bridgeTopupEnabled (ENG-465)#658
Closed
islandbitcoin wants to merge 1 commit into
Closed
feat(flags): gate Bridge top-up/KYC queries behind bridgeTopupEnabled (ENG-465)#658islandbitcoin wants to merge 1 commit into
islandbitcoin wants to merge 1 commit into
Conversation
… (ENG-465) Folds the ENG-465 client kill switch into the cashouts/topups/transfers epic (#621). The Bridge KYC + external-accounts queries fire on mount/focus for every level>=1 user; without a client flag the only way to disable bridge is the backend bridge.enabled, which makes resolvers throw (client errors) rather than degrade gracefully. Add a bridgeTopupEnabled Remote Config flag (default OFF) and Apollo `skip:` the queries + focus refetch so bridge stays dark until flipped on for the beta cohort. USDT gating intentionally omitted: the USDT cutover is the deliberate default (flash#413), so the cashWalletCutover field will exist on main and needs no client skip. ENG-465. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
This was referenced Jun 29, 2026
Contributor
Author
|
Folded into #621 as commit I kept the Remote Config flag idea ( Closing this PR as superseded by #621. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Folds the ENG-465 client kill switch into the #621 epic (
feat/fygaro).What
feature-flags-context.tsx: newbridgeTopupEnabledRemote Config flag (default OFF; on forLocal)TopupCashout.tsx+AccountType.tsx:skip: !bridgeTopupEnabledonuseBridgeKycStatusQuery/useBridgeExternalAccountsQuery+ guard the on-focus refetchWhy
These bridge queries fire on mount/focus for every level≥1 user. With only the backend
bridge.enabledtoggle, disabling bridge makes resolvers throw → client errors. This adds a client-side kill switch so bridge stays dark (graceful) until flipped on for the beta cohort.Scope note
USDT gating is intentionally omitted — the USDT cutover is the deliberate default (flash#413), so
cashWalletCutoverexists on main and needs no client skip. Supersedes the earlier ENG-465 PRs #656/#657, which targeted sibling branches not in this epic.Local typecheck not run (isolated worktree); relying on CI.
🤖 Generated with Claude Code