Content
@
https://opensea.io/collection/evm-6
0 reply
0 recast
2 reactions
nicholas 🧨
@nicholas
Solidity devs! Would anyone be up to take a look at the @crowdfund smart contract? Looking to get more eyes on it before launching a bunch more projects. https://github.com/seedclub/farcaster-crowdfund-contracts/blob/main/src/FarcasterCrowdfund.sol
3 replies
0 recast
6 reactions
maks
@maks
Some minor stuff on very quick review: Why 721 instead of 1155 for commemorative NFT? If using 1155, `tokenId` can equal `crowdFundId` and can kill `tokenToCrowdfund` mapping. 1155 feels more appropriate for use case. Also not sure of reason for `donorToTokenId` mapping, can use donation amount (>0) to check if previously donated. Can use a Bitmap for the bool mappings. Can use named parameters in mappings. Also would consider letting crowdfund creator transfer "ownership" rights. I like the comments "requirements" section. 😀
0 reply
0 recast
0 reaction