feat: isChain composable

[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 isChain composable #6202
• Requires deployment <snek/rubick/worker>

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

• My DOT address: Payout

Copilot Summary

🤖 Generated by Copilot at dd93446

Refactored the chain detection logic by creating a new useIsChain composable that returns computed booleans for each chain based on the prefix. Replaced the isBasilisk variable from the usePrefix composable with the useIsChain composable in the WalletAssetMenu and WalletAssetPortfolio components.

🤖 Generated by Copilot at dd93446

To check the chain of a prefix
We use a composable suffix
useIsChain returns
Booleans that discern
If it's Basilisk, Remark, or AssetHub fix

[kodabot]

SUCCESS @Jarsen136 PR for issue #6202 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	5cee5ad
🔍 Latest deploy log	https://app.netlify.com/sites/koda-canary/deploys/6487cc966a1ec80008cd6560
😎 Deploy Preview	https://deploy-preview-6203--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 introduces a new composable function called useIsChain for checking if the current chain is Basilisk, Remark, or AssetHub. The function is used to refactor WalletAssetMenu.vue and WalletAssetPortfolio.vue components by replacing the previously computed isBasilisk property in the usePrefix composable. The changes involve 4 files, including the addition of the new composable file useIsChain.ts, and the removal of the isBasilisk property from the usePrefix.ts composable. Overall, 17 insertions and 7 deletions have been made to the codebase.

[roiLeo]

I would have just kept them under usePrefix composable, and without arg

[Jarsen136]

I would have just kept them under usePrefix composable, and without arg

WDYT 👀 @vikiival

[vikiival]

Otherwise lgtm

[codeclimate]

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

View more on Code Climate.

[roiLeo]

But wait we already have something like this, I don't find any advatange to have urlPrefix as a param

nft-gallery/composables/usePrefix.ts

Lines 43 to 45 in 09d42f3

	const isBasilisk = computed(
	() => prefix.value === 'bsx' || prefix.value === 'snek'
	)

[vikiival]

The point is why I described to put urlPrefix as param instead of using usePrefix is use-cases such as teleport or future create that would have chain selector.

Moreover two things

1. if we use usePrefix inside the isChain it would mean that many components would call usePrefix twice
2. transaction composables - we pass prefix as the parameter therefore it's simplier to use this via params
3. Separation - there would be some cases where you don't need useIsChain composable and having it inside of usePrefix will add another n properties to compute.

Hope it makes sense now:

[vikiival]

pay 20 usd

[yangwao]

😍 Perfect, I’ve sent the payout
💵 $20 @ 4.68 USD/DOT ~ 4.274 $DOT
🧗 16SjUbGKSdjCdWTy3NNT3JxbRVGGqD4mwkHpc6BD9U2Rp29Z
🔗 0x6260491539781faf8b3f265123421acb6e230cc9fdc40ca007709733be9204ea

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