feat: add retry button to error toast in deposit screen - #524
Conversation
- Extract deposit submission logic into handleSubmitDeposit callback - Show error toast with retry button instead of inline error message - Retry button triggers resubmission of failed transaction - Improved UX for failed transactions with clear recovery path closes Heliobond#2
|
@BABAT-CODE 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! 🚀 |
|
@BABAT-CODE is attempting to deploy a commit to the David Dada's projects Team on Vercel. A member of the Team first needs to authorize it. |
sshdopey
left a comment
sshdopey
left a comment
There was a problem hiding this comment.
Great work on this! Extracting the deposit logic into a reusable callback and adding a retry button to the error toast directly addresses the issue. The UX improvement is clear and the code is well-structured. Keep up the awesome contributions! ❤️
|
Approved! 🎉 I couldn't auto-merge this just yet (it may need a rebase or have a check still running). A maintainer can merge it whenever it's ready. |
dadadave80
left a comment
dadadave80
left a comment
There was a problem hiding this comment.
Please resolve merge conflict
sshdopey
left a comment
sshdopey
left a comment
There was a problem hiding this comment.
Great work on this! Extracting the deposit logic into a reusable callback and adding a retry button to the error toast directly addresses the issue. The code is clean and the UX improvement is clear. I noticed the success toast lost the href link to the investment, so you might want to add that back if it was intentional. Also, the retry button styling uses inline styles, but that's fine for now. Thanks for contributing! ❤️
|
Approved! 🎉 I couldn't auto-merge this just yet (it may need a rebase or have a check still running). A maintainer can merge it whenever it's ready. |
4 participants
closes #427
Summary
Linked issue
Closes #
Type of change
Screenshots / screencast
Checklist
Closes #…)bun run buildpasses locally (builds + type-checks)messages/en.jsonandmessages/fr.json(if any copy changed)