onflow / nft-catalog

https://www.flow-nft-catalog.com/
The Unlicense
36 stars 13 forks source link

Split retrieval of catalog into batches to avoid cadence memory issus and move logic to helper #102

Closed bshahid331 closed 1 year ago

bshahid331 commented 1 year ago

Summary Running into the same issue for the catalog on testnet as we did on proposals for testnet here: https://github.com/dapperlabs/nft-catalog/pull/100

The same short term fix, also I moved the batch logic into a helper function and all uses should just call that and not worry about the batching logic. That should be abstracted out.

Test Plan Confirmed testnet for both catalog and proposals loads.

vercel[bot] commented 1 year ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated
nft-catalog ✅ Ready (Inspect) Visit Preview Nov 21, 2022 at 2:38PM (UTC)