Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feature request: being able to just fetch nft pricing without all the token metadata #82

Open
oveddan opened this issue Jan 20, 2022 · 6 comments

Comments

@oveddan
Copy link

oveddan commented Jan 20, 2022

We just want to be able to fetch nft pricing, without having to fetch all the metadata and token info from opensea. There is no way to do this via the current hooks, but it would be great if this is an option! This is especially important because opensea api is unstable and unreliable, and when it doesnt work (for example now it gives a cors error on our site) the useNFT hook breaks.

So something like:
useNFTPricing would be great!

@iainnash
Copy link
Contributor

Hey!

I'm working on this ability with a custom fetch strategy where you can fetch just metadata, just media info, just market info for each nft or a combination of all thereof as needed.

However, it's a larger refactor and will take some time to roll out across this package and nft-components.

@iainnash
Copy link
Contributor

iainnash commented Apr 4, 2022

Rollout is coming soon!

@0xTranqui
Copy link

@iainnash is the rollout you are mentioning in reference to this more generalized looking nft-metadata repo that's popped up on Zora's Github recently? https://github.com/ourzora/nft-metadata

@iainnash
Copy link
Contributor

iainnash commented Apr 5, 2022

It's in the branch /alpha_impl_hooks for this particular repo

@jake
Copy link

jake commented Jun 3, 2022

was just looking for how to do this as well but having trouble working out how... looks like that branch was deleted and not merged. did that feature end up getting implemented anywhere?

@iainnash
Copy link
Contributor

iainnash commented Jun 3, 2022

Hey! This branch is now merged onto main.

You can just use the standard nft hook data and ignore the nft metadata safely.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants