Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(wallet)_: Assets unsupported for bridge appear as disabled on swap flow #22007

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

smohamedjavid
Copy link
Member

fixes #21959

Summary

This PR fixes assets unsupported for bridge appear as disabled on swap flow.

Review notes

The bug is caused by leftover data (:tx-type) in app-db when the user leaves the bridge flow. We can add clean-up on the un-mount of those pages, but it will be time-consuming to identify as the user can enter from different entry points.
Cleaning up the send data when entering the swap flow is much better.

Platforms

  • Android
  • iOS

Steps to test

  • Open Status
  • Navigate to Wallet Tab
  • Long press on any asset supported for the bridge and enter the bridge flow
  • Navigate back to the home screen
  • Navigate to any account with assets not supported by the bridge
  • Tap on Swap
  • Verify all the assets are shown without disabled state

status: ready

@status-im-auto
Copy link
Member

status-im-auto commented Jan 31, 2025

Jenkins Builds

Commit #️⃣ Finished (UTC) Duration Platform Result
✔️ 5599ac6 #1 2025-01-31 11:38:22 ~6 min tests 📄log
✔️ 5599ac6 #1 2025-01-31 11:39:30 ~7 min android-e2e 🤖apk 📲
✔️ 5599ac6 #1 2025-01-31 11:42:30 ~10 min android 🤖apk 📲
✔️ 5599ac6 #1 2025-01-31 11:43:54 ~11 min ios 📱ipa 📲
✔️ 7985659 #2 2025-02-03 10:23:11 ~4 min tests 📄log
✔️ 7985659 #2 2025-02-03 10:26:30 ~7 min android-e2e 🤖apk 📲
✔️ 7985659 #2 2025-02-03 10:27:09 ~8 min android 🤖apk 📲
✔️ 7985659 #2 2025-02-03 10:29:07 ~10 min ios 📱ipa 📲

@smohamedjavid smohamedjavid force-pushed the fix/clean-send-data-on-entering-swap-flow branch from 5599ac6 to 7985659 Compare February 3, 2025 10:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
request-manual-qa wallet-core Issues for mobile wallet team
Projects
Status: CONTRIBUTOR
Development

Successfully merging this pull request may close these issues.

Unsupported bridge assets appear as disabled on swap pages
2 participants