search

artonomous / artonomous-mvp

A Self-Owning, Self-Sustaining, Self-Improving Autonomous Artist Using the Ethereum Blockchain
158 stars 21 forks source link

upgradeable metadata for NFTs #19

Closed okwme closed 6 years ago

okwme commented 6 years ago

Addresses Issue #17

Adds a separate metadata contract to be used in a delegate call from the erc721 contract. This allows the tokenURI function to return a concatenated string of a base URI and the tokenId (using https://artonomous.ai/metadata/{tokenId}.json for now). This can be replaced with another fully on chain solution of concatenated strings comprising an entire JSON object if desired or be replaced with a contract that implements a more robust metadata standard should one arise in the future.

simondlr commented 6 years ago

I think this makes sense for now. It's okay if the metadata component is upgrade-able for now. This is mostly just to allow others to implement viewers onto the artwork more easily than relying on external parties.

Wondering if others have any opinions?