search

gear-tech / apps

Gear Apps
GNU General Public License v3.0
8 stars 9 forks source link

Add NFT interface #9

Closed LouiseMedova closed 2 years ago

LouiseMedova commented 2 years ago

Resolves # .

@reviewer-or-team

LouiseMedova commented 2 years ago

Approving as I assume we are not implementing exact same thing as of EIP721. EIP721 is very large so it can't be implemented in single PR. Also I believe author of this PR is free to implement NFT taking EIP721 as reference so as far as most of the core features are covered I am happy to approve.

EIP721 is not so large, can you please write what should be implemented from it? I think of adding NFTReceiver and correspondingly safeTransferFrom. And in that interface implementation I didn't add any functions for reading state, but added them in NFT-example.