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.

Sign up for GitHub

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

Jump to bottom

Wallet: Disable complex routing #20869

Closed
J-Son89 opened this issue Jul 24, 2024 · 1 comment · Fixed by #20901
Closed

Wallet: Disable complex routing #20869

J-Son89 opened this issue Jul 24, 2024 · 1 comment · Fixed by #20901
Assignees
@briansztamfater
Labels
feature feature requests high-prio
Milestone
2.30.0

Comments

@J-Son89
Copy link
Contributor

J-Son89 commented Jul 24, 2024

Currently the app does not have the infrastructure to handle complex routing, it brings in many secondary issues and is outright complex to a user.
Since we will be moving away from this by 2.31 it makes sense to block it for the 2.30 release as it is a simple fix to do and will reduce some user confusion if they end up making a multichain transaction.

It can be blocked in
src/status_im/contexts/wallet/send/events.cljs
Screenshot 2024-07-24 at 12 37 23
by updating the best route to be nil if it is a multichain route.
@J-Son89 J-Son89 added high-prio feature feature requests labels Jul 24, 2024
@J-Son89 J-Son89 moved this to Iteration Backlog in Status Desktop/Mobile Board Jul 24, 2024
@briansztamfater briansztamfater self-assigned this Jul 26, 2024
@churik churik added this to the 2.30.0 Beta milestone Jul 26, 2024
@briansztamfater briansztamfater moved this from Iteration Backlog to In Progress in Status Desktop/Mobile Board Jul 26, 2024
@churik
Copy link
Member

churik commented Jul 26, 2024

follow up on #20631

@briansztamfater briansztamfater mentioned this issue Jul 26, 2024
Merged
2 tasks
@github-project-automation github-project-automation bot moved this from In Progress to Done in Status Desktop/Mobile Board Jul 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@briansztamfater briansztamfater

Labels
feature feature requests high-prio
Projects
Status Desktop/Mobile Board
Archived in project
Milestone
2.30.0
Development

Successfully merging a pull request may close this issue.

feat(wallet): disable complex routing status-im/status-mobile
3 participants
@churik @briansztamfater @J-Son89