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

Some questions and proposal #13

Closed
JavierCervilla opened this issue Jan 1, 2022 · 4 comments
Closed

Some questions and proposal #13

JavierCervilla opened this issue Jan 1, 2022 · 4 comments

Comments

@JavierCervilla
Copy link

First of all Happy New Year to all and thanks for this repo, it is helping me to learn a lot, the second thing is that I would like to propose to make a default fetch URI in token creation ERC1155 for those contracts that does not implement the IMETADATAURI Interface because the replacement of the ID is done by the client, on the other hand, I like the system and I have created a fork of this repo because I would like to extend the templates, but I can not make it work, I can not even install the dependencies. Greetings to all and happy 2022.

@Amxx
Copy link
Collaborator

Amxx commented Jan 20, 2022

Hello @JavierCervilla

I'm not sure I fully understand your question, because it's honestly difficult to read, but here are some elements that might provide an answer.

The ERC1155 subgraph indexing is purely based on the specification interface. The base interface described an URI event, and we are using it to populate this subgraph.

It is true that the standard also includes a IERC1155MetadataURI interface, that I've been ignoring so far. I'll definitely add it to the ERC1555 datasource to try and fetch an URI on token creation. This value will be kept, unless overridden by a URI event.

Does that sound good to you ?

@frangio
Copy link
Contributor

frangio commented Jan 20, 2022

From my reading the specification requires a URI event even on token creation, so I don't think it's necessary to add custom handling.

I think OP refers by "replacement of the ID is done by the client" to the feature where the text {id} in the returned URI is replaced with the token id (this is explained in the Metadata section of the EIP). Should the subgraph do this replacement?

@Amxx
Copy link
Collaborator

Amxx commented Jan 20, 2022

its not hard to do it, so why not.

@Amxx
Copy link
Collaborator

Amxx commented Jan 29, 2022

Fixed in #14

@Amxx Amxx closed this as completed Jan 29, 2022
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

3 participants