fix(predict): disable Deposit Wallet withdrawals#29941
Conversation
|
CLA Signature Action: All authors have signed the CLA. You may need to manually re-run the blocking PR check if it doesn't pass in a few minutes. |
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, have a team admin enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit 4a63354. Configure here.
Codecov Report❌ Patch coverage is Additional details and impacted files@@ Coverage Diff @@
## main #29941 +/- ##
=======================================
Coverage 81.54% 81.54%
=======================================
Files 5343 5344 +1
Lines 142128 142146 +18
Branches 32411 32415 +4
=======================================
+ Hits 115894 115910 +16
- Misses 18302 18303 +1
- Partials 7932 7933 +1 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
🔍 Smart E2E Test Selection
click to see 🤖 AI reasoning detailsE2E Test Selection:
Tag selection rationale:
No other tags are warranted — changes are isolated to the Predictions/Polymarket feature with no impact on accounts, networks, swaps, browser, snaps, or other areas. Performance Test Selection: |
|




Description
Temporarily disables the Predict withdraw entry point for Deposit Wallet users until Deposit Wallet withdrawals are fully supported.
Deposit Wallet users with a Predict balance now see a dismissible bottom sheet explaining that withdrawals are currently unavailable and to contact Customer Service for assistance. Legacy/Safe users continue through the existing withdraw flow.
This is intentionally a temporary UI guard and should be removed once Deposit Wallet withdraw support ships.
Changelog
CHANGELOG entry: Fixed Predict withdrawals to show a temporary unavailable message for Deposit Wallet users.
Related issues
Fixes: PRED-869
Manual testing steps
Validation run locally:
yarn lint:tscyarn eslint app/components/UI/Predict/components/PredictWithdrawUnavailableSheet/PredictWithdrawUnavailableSheet.tsx app/components/UI/Predict/components/PredictWithdrawUnavailableSheet/PredictWithdrawUnavailableSheet.test.tsx --cacheyarn jest app/components/UI/Predict/components/PredictWithdrawUnavailableSheet/PredictWithdrawUnavailableSheet.test.tsx --runInBandprinted PASS for all tests, then hit the known local Jest OOM after completion.Screenshots/Recordings
Before
N/A - PR changes the Deposit Wallet withdraw press behavior from launching unsupported withdraw handling to displaying a temporary unavailable bottom sheet.
After
Manually verified in the iOS simulator: Deposit Wallet users see the withdrawals unavailable bottom sheet and can dismiss it with "Got it".
Pre-merge author checklist
Performance checks (if applicable)
trace()for usage andaddTokenfor an exampleFor performance guidelines and tooling, see the Performance Guide.
Pre-merge reviewer checklist
Note
Medium Risk
Changes the withdrawal entry point behavior in the Predict balance card based on detected wallet type, which can block or allow access to a funds-moving flow. Risk is mitigated by added unit/E2E coverage and a conservative default that disables withdraw while account state is unknown.
Overview
Disables Predict withdrawals for Deposit Wallet users by gating the
Withdrawbutton onusePredictAccountStateand routing Deposit Wallet presses to a temporary “withdrawals unavailable” bottom sheet.Adds the new
PredictWithdrawUnavailableSheetcomponent (with i18n strings and test IDs) and wires it intoPredictFeedso it can be opened via a ref callback; Safe/legacy users continue to callwithdraw()as before, and the Withdraw button is disabled until wallet type is resolved.Updates unit tests and Detox/E2E mocks to cover wallet-type branching and keep the withdraw smoke test on the legacy Safe path (new
LEGACY_SAFE_WALLET_ADDRESS+POLYMARKET_LEGACY_SAFE_ACCOUNT_MOCKS).Reviewed by Cursor Bugbot for commit c60a694. Bugbot is set up for automated code reviews on this repo. Configure here.