-
Notifications
You must be signed in to change notification settings - Fork 18
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: Correct 'Use Max' Behavior to Set Input Field and Recalculate Output #142
Conversation
@gabrieltemtsen is attempting to deploy a commit to the Mento Labs Team on Vercel. A member of the Team first needs to authorize it. |
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
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.
Thank you for taking your time to contribute 🦾. Can you change the target branch to dev?
Sure, I have |
Hi @bayological just checking if you're gonna merge the update since it's already approved? 😊 |
…typing in output field
Description
A few sentences describing the overall effects and goals of the pull request's commits.
What is the current behavior, and what is the updated/expected behavior with this PR?
The PR fixes the issue where the "Use Max" button incorrectly sets the output field value after a user types in the output field. The updated behavior ensures the input field is set to the maximum balance and recalculates the output based on the conversion rate when the useMax is triggered.
Other changes
Describe any minor or "drive-by" changes here.
Tested
An explanation of how the changes were tested or an explanation as to why they don't need to be.
Tested manually by entering values in both input and output fields, clicking "Use Max," and confirming the input and output fields behave as expected.
Related issues
Checklist before requesting a review