Skip to content

fix(web): disable deposit and withdraw submit on zero or negative amounts - #686

Open
Otfrugger wants to merge 1 commit into
drydocs:mainfrom
Otfrugger:fix/reject-zero-or-negative-amounts
Open

fix(web): disable deposit and withdraw submit on zero or negative amounts#686
Otfrugger wants to merge 1 commit into
drydocs:mainfrom
Otfrugger:fix/reject-zero-or-negative-amounts

Conversation

@Otfrugger

Copy link
Copy Markdown

Summary

Disables the deposit and withdraw submit buttons when the parsed amount is less than or equal to zero or NaN.

Changes

  • Added zero, negative, and NaN guards (parseFloat(amount) <= 0 || Number.isNaN(parseFloat(amount))) to the submit button disabled condition in both DepositTab and WithdrawTab.
  • Added unit tests in DepositTab.test.tsx and WithdrawTab.test.tsx verifying the button is disabled when amount is '0' or '-1'.

closes #652

@vercel

vercel Bot commented Aug 31, 2026

Copy link
Copy Markdown

@Miracle656 is attempting to deploy a commit to the Collins' projects Team on Vercel.

A member of the Team first needs to authorize it.

@drips-wave

drips-wave Bot commented Aug 31, 2026

Copy link
Copy Markdown

@Otfrugger Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

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 this pull request may close these issues.

[Bug] Deposit/withdraw submit buttons don't reject zero or negative amounts

2 participants