Feature: ERC721 NFT Contract

[julio4]

Description

In this feature, we will implement a simple ERC721 Non-Fungible Token (NFT) contract. This will allow the creation, transfer, and management of NFTs. Users will be able to mint new NFTs, transfer ownership, and query the ownership and metadata of their NFTs.

Criteria:

• Keep it simple, focus on explaining the interface methods with a minimal implementation
• Include comprehensive tests
• Deploy the contract on a testnet

Resources:

• See how we implemented ERC20
• OZ ERC721

ODHack

To be eligible for additional rewards, be sure to review and follow the ODHack Common Guidelines and Contributing Guidelines.
Be sure to join the telegram group and introduce yourself.

[raizo07]

Hello @julio4 I'll like to be assigned to this

[the-first-elder]

i can do this

[Timilehin-bello]

@julio4 I'd like to take on the task of implementing and testing the ERC721 NFT contract. Here's my plan

What I'll Implement:

ERC721 Contract:
Mint new NFTs.
Transfer NFT ownership.
Query ownership and metadata.

Interface Methods:
Minting: Create new NFTs.
Transferring: Transfer ownership of NFTs.
Querying: Get ownership and metadata of NFTs.

Plan:
Implementation:

I will follow the ERC721 standard.
I will use minimal implementation for clarity.
I will refer to OZ ERC721 and our ERC20 for guidance.

Comprehensive Tests:

Test minting functionality.
Test transferring functionality.
Test querying ownership and metadata.
Cover edge cases like invalid transfers and minting.
I will also deploy the contract on a testnet to verify functionality.

Example Test Cases:
Minting:

Ensure new NFTs are created correctly.
Verify metadata is set correctly.

Transferring:

Ensure ownership is transferred correctly.
Handle invalid transfers.

Querying:

Verify correct ownership details.
Check metadata retrieval.

I think this should cover everything needed to make sure the NFT721 contract works correctly. Let me know if I can take this on. Thank you

[BenFaruna]

Hello @julio4 I would like to be assigned to this. I am currently working on an ERC721 contract and I'm about to write tests for it. I can rewrite it to follow the pattern used for the ERC20 contract. I added an image to show what I am currently working on.

Thank you

[coxmars]

Hi, @julio4,

I would like to help with this one if it hasn't been assigned yet. I have extensive experience with Solidity and smart contract development. Here’s my plan:

• Implement the ERC721 NFT contract focusing on minting, transferring, and querying ownership and metadata, leveraging OpenZeppelin's ERC721 for security and compliance.
• Write comprehensive tests to cover all interface methods, ensuring correctness and robustness.
• Deploy the contract on a testnet to validate functionality in a live environment and provide deployment instructions.
• I am confident I can complete this task within 4 days.

Thanks!

[julio4]

Hello @julio4 I'll like to be assigned to this

@raizo07 assigned for now

[julio4]

@raizo07 Any updates on this?

[raizo07]

Yes I'll be sending a PR today.

…

On Tue, 4 Jun 2024, 15:17 Jules Doumeche, ***@***.***> wrote: @raizo07 <https://github.com/raizo07> Any updates on this? — Reply to this email directly, view it on GitHub <#197 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/ATKSYSQL7ILLYRMPN7L6W2LZFXD6JAVCNFSM6AAAAABIMPCYRWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNBXGY2TMMJZGA> . You are receiving this because you were mentioned.Message ID: ***@***.***>

[ScottyDavies]

I am applying to this issue via OnlyDust platform.

My background and how it can be leveraged

I have a background in smart contract development and blockchain technology, with a focus on Solidity programming. I’ve successfully implemented ERC20 tokens and understand the nuances of interacting with blockchain networks.

How I plan on tackling this issue

I would approach this issue with following steps

Design Contract:
Implement a basic ERC721 contract using OpenZeppelin's library to handle NFT creation, transfer, and metadata management.

Interface Method, Focus on core ERC721 methods:

mint(address to, uint256 tokenId): Create new NFTs.
transferFrom(address from, address to, uint256 tokenId): Transfer NFTs between addresses.
ownerOf(uint256 tokenId): Query NFT ownership.
tokenURI(uint256 tokenId): Retrieve metadata.
Write Tests: Develop comprehensive tests to cover minting, transferring, and querying functionality using a framework like Hardhat or Truffle.

Deploy on Testnet:
Deploy the contract on a testnet such as Rinkeby or Goerli, ensuring proper configuration and deployment.