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

Swap issues #2935

Closed
seabornlee opened this issue Nov 11, 2022 · 3 comments · Fixed by #2936
Closed

Swap issues #2935

seabornlee opened this issue Nov 11, 2022 · 3 comments · Fixed by #2936
Assignees

Comments

@seabornlee
Copy link
Contributor

seabornlee commented Nov 11, 2022

For watch only wallet

Now: It does nothing when confirm transaction.
Expected: It shows can't do swap with watch only wallet.

Edit from token balance

Now: It opens Select Route page when editing balance.
Expected: It automatically selects the recommended route and user can open Select Route page by clicking button. Such as 1inch does.

@justindg justindg self-assigned this Nov 11, 2022
@justindg
Copy link
Contributor

justindg commented Nov 11, 2022

For watch only wallet

Now: It does nothing when confirm transaction. Expected: It shows can't do swap with watch only wallet.

In this case, should we allow user to bring up confirmation dialog? Or can we just alert them when they press Continue?

@seabornlee
Copy link
Contributor Author

For watch only wallet

Now: It does nothing when confirm transaction. Expected: It shows can't do swap with watch only wallet.

In this case, should we allow user to bring up confirmation dialog? Or can we just alert them when they press Continue?

I prefer the former so that later other transactions will automatically have the same check.

@justindg
Copy link
Contributor

justindg commented Nov 11, 2022

For watch only wallet

Now: It does nothing when confirm transaction. Expected: It shows can't do swap with watch only wallet.

In this case, should we allow user to bring up confirmation dialog? Or can we just alert them when they press Continue?

I prefer the former so that later other transactions will automatically have the same check.

I only implemented it in the Activity where the transaction happens, since a reference to the Wallet object is needed. I can create new tickets/PRs for other scenarios that need this.

@justindg justindg linked a pull request Nov 11, 2022 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants