-
-
Notifications
You must be signed in to change notification settings - Fork 361
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
Show user balance at minting screen #3473
Conversation
✅ Deploy Preview for koda-nuxt ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
SUCCESS @Jarsen136 PR for issue #3459 which is assigned to you. Please wait for review and don't hesitate to grab another issue in the meantime! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Works for me
Can you rename component to |
I know the issue described only adding the balance so user can see what's wrong, but perhaps we don't want to let user run into the error itself? Can we disable submit buttons on create nfts/create collections, if the balance is 0 with tooltip informing there's a balance issue? wdyt @roiLeo @Jarsen136 I'm still runing into this: |
Even if the balance is great than 0, it may still face this warning `account balance too low.
(btw: This PR is about building a reusable component to show user balance) |
I'm not really a fan of disabled pattern as it can cause some frustrating for the UX. Tooltip component is more of a "Helper" and shouldn't be used to inform user has not enough founds + think about mobile user. (ref #3324, MaterialDesign) |
sure, good to know, whatever works for you and won't result in user seeing that error 😆 |
✅ |
Let's merge this one and open new issue for when user hasn't enough balance to mint. |
pay 100 usd |
😍 Perfect, I’ve sent the payout 🪅 Let’s grab another issue and get rewarded! |
Thank you for your contribution to the KodaDot NFT gallery.
👇 _ Let's make a quick check before the contribution.
PR type
What's new?
Before submitting Pull Request, please make sure:
Optional
Had issue bounty label?
Community participation
Screenshot