perun-network / erdstall-ts-sdk

TypeScript client SDK to interact with Erdstall.
Apache License 2.0
5 stars 2 forks source link

Add metadata provider #81

Closed ndzik closed 2 years ago

ndzik commented 3 years ago

Adds definitions for the NFTMetadataProvider.

Currently blocked.

ndzik commented 2 years ago

Two mentionable points:

  1. I think it would be nice to create a solidity contract defining an "umbrella" interface for metadata stuff. We are currently abusing the fact, that the metadata of all contracts for which metadata is requested, implements the ERC721IMetadata interface. Any opinions?

  2. I fixed a typo in our naming and I am certain it will break stuff in other repositories.