Stuck at claim of Chroma drop #8619

[hassnian]

PR Type

• Bugfix
• Feature
• Refactoring

Context

• Closes Stuck at claim of Chroma drop #8619
• Closes Drops requiring funds to enable auto-teleport #8482
• Closes Check ED on auto teleport  #8526

Did your issue had any of the "$" label on it?

• Fill up your DOT address: Payout

Community participation

• Are you at KodaDot Discord?

Screenshot 📸

• My fix has changed something on UI;

Copilot Summary

copilot:summary

copilot:poem

[kodabot]

WARNING @hassnian PR for issue #8482 which isn't assigned to you. Please be warned that this PR may get rejected if there's another assignee for issue #8482

[kodabot]

WARNING @hassnian PR for issue #8526 which isn't assigned to you. Please be warned that this PR may get rejected if there's another assignee for issue #8526

[netlify]

✅ Deploy Preview for koda-canary ready!

Name	Link
🔨 Latest commit	f14e663
🔍 Latest deploy log	https://app.netlify.com/sites/koda-canary/deploys/657dbae36da8fd0008dce15d
😎 Deploy Preview	https://deploy-preview-8624--koda-canary.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 configuration.

[kodabot]

WARNING @hassnian PR for issue #8619 which isn't assigned to you. Please be warned that this PR may get rejected if there's another assignee for issue #8619

[yangwao]

I see conflicts @hassnian happy to test!

[reviewpad]

AI-Generated Summary: This pull request addresses a bug where a modal was getting stuck in the application. In the 'AddFundsModal.vue' and 'AutoTeleportActionButton.vue' components, the developer modifies the condition for setting the loading state and showing add funds methods respectively, by checking for the isReady property instead of hasBalances of the autoteleport data object. This makes the UI dependent on the readiness of the autoteleport data instead of the presence of its balances data.

Furthermore, the developer makes changes to the useAutoTeleportTransitionDetails.ts composable function, to include a balances boolean in the fetched object. This boolean is false when we start fetching the transition balances and true when it ends - consequently affecting the hasBalances computed property which now relies on the balances having been fetched.

Overall, the changes in this pull request ensure that certain parts of the UI are updated based on the readiness of autoteleport data and completion of the fetching of transition balances.

[yangwao]

ok worked for me tho.

we can extend it with mint if so then? for follow up I guess

[yangwao]

@hassnian let's resolve conflicts if you figure it out, it worked for me! 😄

[codeclimate]

Function useDropMinimumFunds has 29 lines of code (exceeds 25 allowed). Consider refactoring.

[codeclimate]

Code Climate has analyzed commit f14e663 and detected 1 issue on this pull request.

Here's the issue category breakdown:

Category	Count
Complexity	1

View more on Code Climate.

[sonarcloud]

Quality Gate passed

Kudos, no new issues were introduced!

0 New issues
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

[yangwao]

Thanks!
pay 30 usd

[yangwao]

😍 Perfect, I’ve sent the payout
💵 $30 @ 7.2 USD/DOT ~ 4.167 $DOT
🧗 13QUj3pZyFNfYj4AM336hRdyLQbevj5H3sR4PKmLEXLdwZhh
🔗 0xae5c0a58b2a752e403c6a4e485542e8cdb9889682296881c14cb3396549d43cc

🪅 Let’s grab another issue and get rewarded!
🪄 github.com/kodadot/nft-gallery/issues