fix: unit display on snek network

[Jarsen136]

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

• Bugfix
• Feature
• Refactoring

Context

• Closes fix the unit display on snek item #5819
• Requires deployment <snek/rubick/worker>

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

Optional

• I've tested it at </ksm/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.

Copilot Summary

🤖 Generated by Copilot at 4408de1

Removed hardcoded value from getKusamaAssetId function in utils/api/bsx/query.ts. This allows the function to return the correct asset id for the Kusama network based on the input network name.

🤖 Generated by Copilot at 4408de1

No more return 5
getKusamaAssetId works
With networks object

[kodabot]

SUCCESS @Jarsen136 PR for issue #5819 which is assigned to you. Please wait for review and don't hesitate to grab another issue in the meantime!

[netlify]

✅ Deploy Preview for koda-canary ready!

Name	Link
🔨 Latest commit	d7706a7
🔍 Latest deploy log	https://app.netlify.com/sites/koda-canary/deploys/644d498563bf6c0008a315a2
😎 Deploy Preview	https://deploy-preview-5821--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 settings.

[reviewpad]

AI-Generated Summary: This pull request contains a change in the utils/api/bsx/query.ts file, fixing the unit display on the snek network by removing a superfluous return statement. The code now returns the correct 'bsx' and 'rmrk' cases, skipping the unnecessary 'snek' case.

[roiLeo]

Not sure about that one:

    // '0' if paying in bsx (on Basilisk and Snek)
    // '1' if paying in ksm (on basilisk)
    // '5' if paying in ksm (on Snek)

[Jarsen136]

Not sure about that one:

    // '0' if paying in bsx (on Basilisk and Snek)
    // '1' if paying in ksm (on basilisk)
    // '5' if paying in ksm (on Snek)

Could you pls review this? @vikiival 👀

[vikiival]

Yes this is still correct

[vikiival]

This would point to Phala token on snek

[Jarsen136]

Query assetList does not return the asset id of 5, that's why we have the unit display. So I think it's a resolver issue related to snek squid side.

[vikiival]

SNEK (rococo)

https://squid.subsquid.io/snekk/v/004/graphql

BSX

https://squid.subsquid.io/snekk/v/005/graphql

[codeclimate]

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

View more on Code Climate.

[Jarsen136]

SNEK (rococo)

https://squid.subsquid.io/snekk/v/004/graphql

BSX

https://squid.subsquid.io/snekk/v/005/graphql

Thanks for checking the query. Then I got the root reason why it does not work. Please review the code again.

[vikiival]

Nice one

[roiLeo]

Wfm, I think I'll remove this plugin in #5655

[roiLeo]

Hello, something might be wrong with this change (tested on /snek/create)

[vikiival]

This would point to Phala token on snek

cc @roiLeo

[vikiival]

Hello, something might be wrong with this change

My bet is that someone changed the token number for snek (propably check assetList)

[roiLeo]

My bet is that someone changed the token number for snek (propably check assetList)

I think I made a change recently on Money component that might be related

edit: issue with this.$route.params.prefix from PrefixMixin