🐛 name could be null

[vikiival]

PR Type

• Bugfix
• Feature
• Refactoring

Context

• Closes #<issue_number>
• 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 📸

Copilot Summary

🤖 Generated by Copilot at 765dca1

Fixed a bug that caused errors when sorting collections by popularity. Used optional chaining on a.meta.name in usePopularCollections.ts.

🤖 Generated by Copilot at 765dca1

Sorting collections
a.meta.name may be nil
Use optional chain

[netlify]

✅ Deploy Preview for koda-canary ready!

Name	Link
🔨 Latest commit	765dca1
🔍 Latest deploy log	https://app.netlify.com/sites/koda-canary/deploys/645cfc9f8ae3e40008a8efd8
😎 Deploy Preview	https://deploy-preview-5924--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 addresses a bug where the name property could be null. A change has been made in the usePopularCollections.ts file to handle this case by adding an optional chaining operator before the localeCompare method. This ensures that the sorting function does not throw an error when the name is null.

[codeclimate]

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

View more on Code Climate.

[reviewpad]

Reviewpad Report

⚠️ Warnings

• Please link an issue to the pull request