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.

Sign up for GitHub

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

Jump to bottom

feat: show a warning if the user has insufficient balance for the tra… #645

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

feat: show a warning if the user has insufficient balance for the tra… #645

wants to merge 2 commits into from

Conversation

andreabadesso
Copy link
Contributor

@andreabadesso andreabadesso commented Jan 10, 2025 •
edited
Loading

ScreenRecording_01-10-2025.19-00-29_1.MP4

Acceptance Criteria

  • We should display an insufficient balance error if the wallet has no available funds to fulfill the deposits for a given nano contract.

Security Checklist

  • Make sure you do not include new dependencies in the project unless strictly necessary and do not include dev-dependencies as production ones. More dependencies increase the possibility of one of them being hijacked and affecting us.

@andreabadesso andreabadesso force-pushed the feat/show-insufficient-balance branch from bb36407 to 3cb0d0e Compare January 10, 2025 21:53
@andreabadesso andreabadesso force-pushed the feat/show-insufficient-balance branch from 3cb0d0e to 35fd498 Compare January 10, 2025 21:59
@andreabadesso andreabadesso self-assigned this Jan 10, 2025
@andreabadesso andreabadesso added the enhancement New feature or request label Jan 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@pedroferreira1 pedroferreira1 pedroferreira1 approved these changes

@tuliomir tuliomir tuliomir approved these changes

Assignees

@andreabadesso andreabadesso

Labels
enhancement New feature or request
Projects
Hathor Network
Status: In Review (Done)
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@andreabadesso @tuliomir @pedroferreira1