gallery optimization

[JKrupinski]

To avoid too many requests I had to change pausing gifs to use nft type property (which is mime type)

It works but looks like this attribute is added only for the newest NFTs

Second thing is that Money component makes too much requests to coingecko on gallery page, I've changed it to use component lazy loading and now it sends requests to coingecko only when NFT is visible

But I have another idea, maybe we can change Money component to get KSM price only once and save it in vuex?
So there will be only one request to coingecko on gallery page

[JKrupinski]

• pausing gifs is back (without any requests, it will work only for nfts with metadata nft.type)
• request for ksm price is executed only once (page load) and fiat price is saved to vuex

@vikiival still not sure about formatBalance decimals, I saw that you changed it to 2
84f8bc0#diff-e5a8b28ff23cd499f3f7efb280de392a5175628ec1785f2e7653e91a0e41adb9R7

but it doesn't work properly, it shows for example: 4.9000 KSM / 21,520,800,000,000.0000 USD so I reverted it back to formatBalance(decimals, showFiatValue.toUpperCase()) for now

[vikiival]

@vikiival still not sure about formatBalance decimals, I saw that you changed it to 2

Yeah It's highly possible, I think it didn't work on gallery

Meanwhile it was fixed by acc45df

[JKrupinski]

@yangwao please have a test and merge if everything looks good

[yangwao]

Some are paused, some are not. https://deploy-preview-316--nftkodadot.netlify.app/rmrk/gallery

Ideas why? Maybe types you are checking for?

I'm stoked to have paused gifs back again :D

Screen.Recording.2021-05-11.at.10.57.44.mov

Screen.Recording.2021-05-11.at.10.56.58.mov

[JKrupinski]

Some are paused, some are not. https://deploy-preview-316--nftkodadot.netlify.app/rmrk/gallery

Ideas why? Maybe types you are checking for?

I'm stoked to have paused gifs back again :D

Screen.Recording.2021-05-11.at.10.57.44.mov
Screen.Recording.2021-05-11.at.10.56.58.mov

As I wrote in previous comment it will work only for nfts with metadata nft.type (which is mime type). Not sure why some of them don't have it, any ideas?

[yangwao]

Maybe @vikiival can chime on this. I don't know either, I mean eventually we want to migrate to subquery as it offer more room to grow :)

[vikiival]

Not sure why some of them don't have it, any ideas?

They were lost in the echo.
Honestly I don't know, I can remigrate those without type.