5316 payments

[vikiival]

It's really 💩 fix but should work

What was happening?

From url we checked usd amount, multiplied with ksm value (which is 0 when component is created)
the result of multiply was Infinity and that failed to sign the transaction.

• 🐛 making props in disabled input optional
• 🏷️ properly typed
• 💩 resolve usd value to KSM

Thank you for your contribution to the KodaDot NFT gallery.

👇 _ Let's make a quick check before the contribution.

PR Type

• Bugfix
• Feature
• Refactoring

Context

• Closes Transfer doesn't trigger signer window  #5316
• Requires deployment <>

Before submitting pull request, please make sure:

• My contribution builds clean without any errors or warnings
• I've merged recent default branch -- main and I've no conflicts
• I've tried to respect high code quality standards
• I've didn't break any original functionality
• I've posted a screenshot of demonstrated change in this PR

Optional

• I've tested it at </bsx/collection>
• I've tested PR on mobile
• I've written unit tests 🧪
• I've found edge cases

Had issue bounty label?

• Fill up your KSM address: Payout

Community participation

• Are you at KodaDot Discord?

Screenshot 📸

• My fix has changed something on UI; a screenshot is best to understand changes for others.

BETA:

PR:

[netlify]

✅ Deploy Preview for koda-nuxt ready!

Name	Link
🔨 Latest commit	17373b5
🔍 Latest deploy log	https://app.netlify.com/sites/koda-nuxt/deploys/6419d925f3a6a70008bc3f60
😎 Deploy Preview	https://deploy-preview-5317--koda-nuxt.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

To edit notification comments on pull requests, go to your Netlify site settings.

[roiLeo]

isn't possible without timeout? maybe watcher

[Jarsen136]

isn't possible without timeout? maybe watcher

I agree.

Or we could try something like this, to update data after fetching the ksm price.

this.fiatStore.fetchFiatPrice().then(() => {
  this.onUSDFieldChange()
})

[vikiival]

By @roiLeo via watcher in 2fc4741

By @Jarsen136 via then in 994effc

[codeclimate]

Code Climate has analyzed commit 17373b5 and detected 0 issues on this pull request.

View more on Code Climate.

[yangwao]

seems not sure if it worked for someone, but it's stuck for me

• Show finalised and best block to transfer #5319