feat(flags): gate cashWalletCutover query behind usdtWalletEnabled (ENG-465)#657
Closed
islandbitcoin wants to merge 1 commit into
Closed
feat(flags): gate cashWalletCutover query behind usdtWalletEnabled (ENG-465)#657islandbitcoin wants to merge 1 commit into
islandbitcoin wants to merge 1 commit into
Conversation
…NG-465) The CashWalletCutover modal queries cashWalletCutover on the home screen for every account. If the app ships before the backend exposes that field, users hit GraphQL errors. Add a usdtWalletEnabled Remote Config flag (default OFF) and Apollo `skip:` the query when off. Modal already hides on null data, so OFF is a clean no-op. ENG-465. Mirrors the existing deviceAccountEnabled skip pattern. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
This was referenced Jun 29, 2026
Contributor
Author
|
Superseded by #658, which folds the same bridge kill-switch into the #621 epic (feat/fygaro). #656/#657 targeted sibling branches (fix/bridge-kyc-selfie, feat/usdt-display-as-usd) that are not part of the epics being merged. USDT query gating dropped: the USDT cutover is the intended default (flash#413). |
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.
What
Adds a
usdtWalletEnabledFirebase Remote Config flag (default OFF) and gates the USDT cutover query behind it.feature-flags-context.tsx: newusdtWalletEnabledflag (default off; on forLocal)CashWalletCutoverModal.tsx:useCashWalletCutoverQuery({ skip: !usdtWalletEnabled })Why (ENG-465, audit P0)
cashWalletCutoveris queried from the home screen for every account. If the app reaches users before the backend exposes the field, they hit GraphQL validation errors. The modal already hides when the data is null, so with the flag OFF this is a clean no-op.Toggle behavior
Verification
Local full typecheck not run (isolated worktree has no node_modules) — relying on CI. Minimal change following the existing
skip: !deviceAccountEnabledpattern.🤖 Generated with Claude Code