fix: You can change the network field to any string you want

[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

Needs QA check

• @kodadot/qa-guild please review

Context

• Closes You can change the network field to any string you want #6563
• Requires deployment <snek/rubick/worker>

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

• My DOT address: Payout

Screenshot 📸

• My fix has changed UI

Copilot Summary

🤖 Generated by Copilot at cbd5ffb

Improved prefix selection logic in usePrefix.ts by validating the route parameter and using it as the first option.

🤖 Generated by Copilot at cbd5ffb

validPrefixFromRoute
checks the list, avoids errors
autumn of bugs ends

[kodabot]

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

[reviewpad]

AI-Generated Summary: This pull request introduces changes to the function within composables/usePrefix.ts. In particular, the patch adds a new constant validPrefixFromRoute that checks whether the prefix from the route parameters is available in the prefixes list. This new constant is then used in the definition of the prefix computed property, replacing the previous reference to route.params.prefix. This modification ensures that the network field can take on any string value from the whitelist.

[netlify]

✅ Deploy Preview for koda-canary ready!

Name	Link
🔨 Latest commit	c400cc3
🔍 Latest deploy log	https://app.netlify.com/sites/koda-canary/deploys/64cfa3343e25be00090d71ad
😎 Deploy Preview	https://deploy-preview-6569--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.

[vikiival]

With this fix you can just check if initial is not undefined

[Jarsen136]

With this fix you can just check if initial is not undefined

Sorry, I'm not sure what do you mean. This fix is not about checking if it's undefined, but checking if the prefix params on route is valid.

before:

Visit: https://canary.kodadot.xyz/hello

after fix:
Visit: https://deploy-preview-6569--koda-canary.netlify.app/hello

[vikiival]

What I wanted to say is that these two lines are very similar.

I wanted to ask if its possible to somehow optimize it.

Otherwise lgtm

[Jarsen136]

What I wanted to say is that these two lines are very similar.

I wanted to ask if its possible to somehow optimize it.

Yes, I have updated it.

[codeclimate]

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

View more on Code Climate.

[sonarcloud]

Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

No Coverage information
2.3% Duplication

[yangwao]

pay 50 usd

[yangwao]

😍 Perfect, I’ve sent the payout
💵 $50 @ 4.99 USD/DOT ~ 10.02 $DOT
🧗 16SjUbGKSdjCdWTy3NNT3JxbRVGGqD4mwkHpc6BD9U2Rp29Z
🔗 0x7d8e3081f9f32cb828ab0d0f3b71a0fca5a2906ad42f6451658978cdf696afd4

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