-
-
Notifications
You must be signed in to change notification settings - Fork 353
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
fix: Zero balance on transfer when address has DOTs #7687
Conversation
✅ Deploy Preview for koda-canary ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
Code Climate has analyzed commit efb061f and detected 0 issues on this pull request. View more on Code Climate. |
AI-Generated Summary: This pull request includes a fix for the issue where the balance becomes zero on transfer when the address contains DOTs. Changes have been made to the Transfer.vue file where a new function called updateAuthBalance has been introduced. This function updates the balance of the given account ID. The trigger for this function is the change in urlPrefix or at the moment of component mounting. In addition to that, the calculation of the transaction fee has been kept inside onMounted lifecycle hook as well. The useIdentityStore has been used for maintaining user identity details. |
Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information |
do they load? yes |
This PR is mainly solving the issue that the balance would not show when We visit the transfer page directly through URL, without expanding the sidebar.
related with #7670
related with #7675 |
yeah i know i know, thanks for linking the issues! AssetHubs are taking +- 30 seconds to show balance initially, different from the other chains that take +- 5 seconds |
thanks ! pay 20 usd |
😍 Perfect, I’ve sent the payout 🪅 Let’s grab another issue and get rewarded! |
Thank you for your contribution to the KodaDot - One Stop Shop for Polkadot NFTs.
👇 __ Let's make a quick check before the contribution.
PR Type
Needs QA check
Context
Did your issue had any of the "$" label on it?
Screenshot 📸
Copilot Summary
🤖 Generated by Copilot at efb061f
Added identity and balance management to
Transfer.vue
component. The component now usesuseIdentityStore
to access and update the user's identity and balance on different networks and accounts.🤖 Generated by Copilot at efb061f