-
Notifications
You must be signed in to change notification settings - Fork 80
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
feat: allow Eth selection in the send flow #4242
Conversation
…wallet into jophish/multi-chain-balances
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks close!
@@ -64,9 +64,9 @@ function useTransactionCallbacks({ | |||
inputAmount: inputIsInLocalCurrency ? localAmount! : tokenAmount, | |||
tokenAmount, | |||
amountIsInLocalCurrency: inputIsInLocalCurrency, | |||
tokenAddress: transferTokenAddress, | |||
tokenAddress: tokenInfo!.address!, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
bumping this
Co-authored-by: Charlie Andrews-Jubelt <cajubelt@users.noreply.github.com>
This reverts commit 63489ef.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🥳 🥳 🥳
# Conflicts: # src/tokens/selectors.test.ts
### Description This was added in #4242 when we were working towards multichain, but now there shouldn't be a need for multiple TokenBalance types ### Test plan CI ### Related issues N/A ### Backwards compatibility Yes ### Network scalability If a new NetworkId and/or Network are added in the future, the changes in this PR will: - [x] Continue to work without code changes, OR trigger a compilation error (guaranteeing we find it when a new network is added)
Description
Allows the selection of Eth in the send flow and the use of the Max button while preserving functionality of the existing Celo send flow.
Video
Screen.Recording.2023-09-29.at.3.28.02.PM.mov
Test plan
Prod Multichain Testers
segment in Statsig.Related issues
Backwards compatibility
Yes