-
-
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
fix: Teleport bridge is showing incorrect balances #6446
Conversation
✅ Deploy Preview for koda-canary ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
AI-Generated Summary: This pull request introduces changes to the 'Teleport.vue' file in the 'teleport' component to fix incorrect balance issues. The changes involve using the 'nativeBalance' property instead of the 'balance' property for multiBalances of several chains including KUSAMA, BASILISK, and STATEMINE. Also, a modification in the 'myKsmBalance' computed property is noted where the returned balance value is no longer multiplied by 'ksmTokenDecimals.value'. The patch includes 6 insertions and 4 deletions. |
Code Climate has analyzed commit 219f99a and detected 0 issues on this pull request. View more on Code Climate. |
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.
✅ code lgtm
Welcome! @helloitsdamsky could you check using https://deploy-preview-6446--koda-canary.netlify.app and tell us if it is fixed for you? |
Pinged Damsky through DM |
everything seems fine! Only.. in one instance we are using , and in other instance we are using . I’d like to see it consistent, using DOT (.) e.g. 6.9755 |
That's more how component handles it vs how we handle displaying currencies |
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
Context
Did your issue had any of the "$" label on it?
Community participation
Screenshot 📸
/rmrk/teleport
Copilot Summary
🤖 Generated by Copilot at 219f99a
Improved KSM balance display in
Teleport.vue
component. UsednativeBalance
for cross-chain compatibility and simplifiedbalance
calculation.🤖 Generated by Copilot at 219f99a