fix: image previews on moonriver

[preschian]

Thank you for your contribution to the KodaDot NFT gallery.
👇 _ Let's make a quick check before the contribution.

PR type

• Bugfix
• Feature
• Refactoring

What's new?

• PR closes Image previews on Moonriver #4020
• PR closes Replace main Pinata gateway with Kodadot pinata instance on IPFS #3952
• PR closes Issue with indexedDB #3338

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
• I've posted a screenshot of demonstrated change in this PR

Optional

• I've tested it at </rmrk/collection/26902bc2f7c20c546a-1FVG7>
• I've tested PR on mobile and everything seems works
• I found edge cases
• I've written some unit tests 🧪

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.

[kodabot]

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

[kodabot]

SUCCESS @preschian PR for issue #4020 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-nuxt ready!

Name	Link
🔨 Latest commit	6dedfb8
🔍 Latest deploy log	https://app.netlify.com/sites/koda-nuxt/deploys/6332eb1e4a70650008c89f47
😎 Deploy Preview	https://deploy-preview-4027--koda-nuxt.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.

[preschian]

some collection image was still using a placeholder because the metadata was null, maybe we need to fix that from the indexer?

and, I don't know how to fix this deepsource issue https://deepsource.io/gh/kodadot/nft-gallery/run/403d3893-71a4-4cf4-92a0-0882e1129d9a/javascript/JS-0336?listindex=1

another error from deepsource Detected the use of variables before they are defined. should I fix that in this PR or in the separate PR? I think we should put the eslint rule for that https://eslint.org/docs/latest/rules/no-use-before-define

[petersopko]

yeah wfm, except for this:

some collection image was still using a placeholder because the metadata was null, maybe we need to fix that from the indexer?

perhaps related to this created issue

• How to handle collection metadata click#34

I think this is good solution until the indexer get's fixed?

[preschian]

perhaps related to this created issue

• How to handle collection metadata click#34

I think this is good solution until the indexer get's fixed?

I see, yes I think so. I just realized this PR should fix this issue also #3338
that error occurs when we set idb-keyval with a null as a key, for example: get(null, data)

[Jarsen136]

small stuff

[petersopko]

@preschian let's check that deepsrouce issue and other than that, I think let's merge this to have some previews ready

[petersopko]

lgtm

[Jarsen136]

code lgtm

[petersopko]

pay 50 usd

[codeclimate]

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

Here's the issue category breakdown:

Category	Count
Complexity	1

View more on Code Climate.

[yangwao]

😍 Perfect, I’ve sent the payout
💵 $50 @ 45 USD/KSM ~ 1.111 $KSM
🧗 DY4SQF2iD456tH89aQtz5wv1EV3BbSW8wKKuMcwbmXaj1pM
🔗 0x506e82136b2a226545f739f2ff511f82cbb70fb8083050ae2f1483e9a79ffd4c

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

[petersopko]

looks like auto-merge doesn't care about some of the deepsource issues which jumped out again after merging main into this, it doesn't look that serious, let's catch it elsewhere