From 11400e16f7384b1c4f5b403ce58eaca19fe0887a Mon Sep 17 00:00:00 2001 From: Theodosis Aggelis Date: Fri, 17 Apr 2020 22:09:24 +0300 Subject: [PATCH] Remove misleading instuction tokenURI should not be needed and could confuse students, tokenURI is already defined in the contract before ERC721Metadata --- eth-contracts/contracts/ERC721Mintable.sol | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/eth-contracts/contracts/ERC721Mintable.sol b/eth-contracts/contracts/ERC721Mintable.sol index e4c3d1a73..dc043dd96 100644 --- a/eth-contracts/contracts/ERC721Mintable.sol +++ b/eth-contracts/contracts/ERC721Mintable.sol @@ -456,7 +456,7 @@ contract ERC721Metadata is ERC721Enumerable, usingOraclize { // - make the base token uri: https://s3-us-west-2.amazonaws.com/udacity-blockchain/capstone/ // 2) create a public mint() that does the following: // -can only be executed by the contract owner -// -takes in a 'to' address, tokenId, and tokenURI as parameters +// -takes in a 'to' address and tokenId as parameters // -returns a true boolean upon completion of the function // -calls the superclass mint and setTokenURI functions