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

Hestia - Limit buy order form validation is done incorrectly for the amount field stopping orders from being placed #631

Open
jaymansfield opened this issue Mar 14, 2024 · 0 comments · Fixed by Polkadex-Substrate/Polkadex-Orderbook-Frontend#1145

Comments

@jaymansfield
Copy link

jaymansfield commented Mar 14, 2024

Limit buy order form validation is done incorrectly for the amount field

image

It seems to error if the amount of pdex you are requesting is greater then your USDT balance (it should compare the total instead)

Screen Shot 2024-03-14 at 3 42 27 PM

When reducing the amount of pdex to be lesss then your USDT balance it accepts it

Screen Shot 2024-03-14 at 3 42 32 PM

onchainaddress:{esm99RRxKvainDsRR6UHRcju4vNnAw1ouZEw64Yfn12tc6HpQ}

@jaymansfield jaymansfield changed the title Hestia - Buy order form validation is wrong - Can't buy more pdex then you already own Hestia - Buy order form validation is wrong Mar 14, 2024
@jaymansfield jaymansfield changed the title Hestia - Buy order form validation is wrong Hestia - Limit buy order form validation is wrong Mar 14, 2024
@jaymansfield jaymansfield changed the title Hestia - Limit buy order form validation is wrong Hestia - Limit buy order form validation is done incorrectly for the amount field Mar 14, 2024
@ap211unitech ap211unitech self-assigned this Mar 15, 2024
@jaymansfield jaymansfield changed the title Hestia - Limit buy order form validation is done incorrectly for the amount field Hestia - Limit buy order form validation is done incorrectly for the amount field stopping orders from being placed Mar 15, 2024
@ap211unitech ap211unitech linked a pull request Mar 18, 2024 that will close this issue
8 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants