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

Fallback to getting collectible contract information from the contract itself. #686

Open
fmdev99 opened this issue Feb 2, 2022 · 3 comments

Comments

@fmdev99
Copy link

fmdev99 commented Feb 2, 2022

Currently the controller is getting collectible contract name and symbol information from the contract itself, but all other information is retrieved from OpenSea. As a fallback, metamask could pull information directly from the contract if OpenSea doesn't have the data. This could use the OpenSea standard here for "contractURI": https://docs.opensea.io/docs/contract-level-metadata

A big motivation for this is currently only projects on the 3 chains supported by OpenSea can have icons for their NFT collections in metamask mobile. This would provide a mechanism for projects that aren't on a chain support by OpenSea to specify an image icon for their NFT collections.

@mcmire
Copy link
Contributor

mcmire commented Mar 23, 2022

@adonesky1 Can you answer this one? I feel like the changes you've made recently could possibly address this.

@adonesky1
Copy link
Contributor

This would involve us endorsing the addition of a new method not included in either the ERC721 or ERC1155 contract standard, one that OpenSea has unilaterally defined. I'm not against this method per se but I think we should discuss before defaulting into adoption of this new standard proposed by OpenSea. In any case this would only apply to contracts that have already adopted this standard recommended to them by OpenSea or any contracts that elect to do so in the future. cc @mcmire @FrederikBolding @rekmarks

@fmdev99
Copy link
Author

fmdev99 commented Dec 20, 2022

This would involve us endorsing the addition of a new method not included in either the ERC721 or ERC1155 contract standard, one that OpenSea has unilaterally defined. I'm not against this method per se but I think we should discuss before defaulting into adoption of this new standard proposed by OpenSea. In any case this would only apply to contracts that have already adopted this standard recommended to them by OpenSea or any contracts that elect to do so in the future. cc @mcmire @FrederikBolding @rekmarks

But the current method is literally only pulling information from OpenSea. So, you already only support information unilaterally defined by OpenSea. By supporting this standard, you at least open the option of other chains working which OpenSea doesn't support.

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

No branches or pull requests

4 participants