added hardhat project for pixel avatars

[etherinsight]

This is a working version of the hardhat solidity project for pixel avatars.

[vercel]

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/developdao/pixel-avatars/DejHYrapoYEpConJ9hVnie2mbJL5
✅ Preview: https://pixel-avatars-git-fork-etherinsight-main-developdao.vercel.app

[etherinsight]

This PR contains:

• Solidity contract for pixel avatars

• hardhat config to
  - run the project in rinkeby testnet and
  - verify contract on etherscan

• readme file with instructions

[Dhaiwat10]

Thanks for putting this in @etherinsight! I was wondering why is it that you chose not to import the source code for these utility contracts: ERC721Enumerable, ReentrancyGuard, Ownable.

(I'm not experienced w/ mainnet contract development so let me know if this is a common practice for mainnet/prod)

[etherinsight]

@Dhaiwat10 I used a flattened code file as my reference which has all the required modules ERC721Enumerable, ReentrancyGuard, Ownable in that single solidity file.

A command like this will create a single file including all imports like ERC721, Ownable etc

npx hardhat flatten contracts/unflattened.sol >flatttened.sol

[Dhaiwat10]

Gotcha. I got to read up on this later and found out that flattened contracts are required(?) if you want your contract to be 'verified' on Etherscan. Is that correct?

[Dhaiwat10]

Else, this looks good to me. Let's get this merged and add tests?

[Dhaiwat10]

Why is this payable tho?

[etherinsight]

Reposting my comment as etherinsight again 😅

good catch. Not sure why this was kept payable on my reference contract. I'll remove this.

[Dhaiwat10]

Haha cool