fix: set minimum amount for Ethereum network#349
fix: set minimum amount for Ethereum network#349Dprof-in-tech wants to merge 4 commits intostablefrom
Conversation
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
app/pages/TransactionForm.tsx
Outdated
| const errorMessage = cngnRateError || "No available quote"; | ||
| setRateError(errorMessage); | ||
| } | ||
| } else if (selectedNetwork?.chain?.name === "Ethereum") { |
There was a problem hiding this comment.
@Dprof-in-tech if (selectedNetwork?.chain?.name === "Ethereum") and (normalizedToken === "CNGN"), this still assumes that the minimum will be 50 for CNGN and other respective local currencies.
Within this if block, let's add a check for when the selected token is CNGN
|
kindly check for balance first. |
balance check follows existing method where we check is it above the minimum transaction amount? then we check if its below your balance |
Description
This pull request introduces a network-specific validation to the transaction form, ensuring users meet the minimum transaction amount for Ethereum.
Validation update for Ethereum transactions:
TransactionForm.tsx, a $50 USD minimum is enforced when the selected network is Ethereum.References
WE ARE ENFORCING A $50 USD MINIMUM FOR ETHEREUM
Testing
Checklist
mainBy submitting a PR, I agree to Paycrest's Contributor Code of Conduct and Contribution Guide.