Integrate NFT Minting on Land Approval in LandRegistry Contract

[onlydustapp]

📚 Overview

When a land is approved in the LandRegistry contract, an NFT should be automatically minted. This NFT will represent the approved land and include all relevant metadata. This feature update will ensure that the NFT minting process is seamlessly integrated into the land approval step.

🛠️ Tasks & Requirements

1. Review Current Logic

   • Examine the existing registration and approval flow in the LandRegistry contract.

2. Integrate NFT Minting

   • Modify the approval process to trigger the minting of an NFT upon successful approval.
   • Ensure that the NFT contains accurate and complete metadata tied to the approved land.

3. Smart Contract Updates

   • Update or refactor any associated smart contract modules to support NFT minting.
   • Emit appropriate events related to the minting process for on-chain verification.

4. Testing

   • Update all relevant unit and integration tests to cover the new flow.
   • Verify that the NFT minting and metadata setup can be confirmed via both on-chain interactions and emitted events.

⚙ Technical Considerations

• Confirm all NFT metadata fields mirror the land's details (e.g., coordinates, area, land use type, etc.).
• Ensure compatibility with the current LandVer system on Starknet (reference: README).
• Update comments to clearly explain the changes in the flow.

✅ Definition of Done

• The LandRegistry contract mints an NFT with correct metadata when a land is approved.
• All related events are emitted as expected for the minting action.
• Tests are updated and passing for both unit and integration scenarios, verifying the updated flow.
• Documentation and code comments clearly explain the new NFT minting process.

Let's ensure this enhancement aligns with LandVer's goals of secure and transparent land verification. 🚀

[SoarinSkySagar]

I am an experienced Cairo contract developer since 2024 and would like to take up this issue

eta: 2 days

[big14way]

please assign me this issue i can handle this
i have done it before

[Mrwicks00]

I'm interested in taking on this issue. I have experience working with smart contract languages such as Solidity and Cairo, and I am familiar with the broader blockchain development ecosystem.

I'm drawn to this issue not only because it aligns with my technical skills, but also because I see it as an opportunity to contribute meaningfully to the project while continuing to grow as a developer.

I'm confident I can handle this task effectively and would appreciate the opportunity to contribute.

[Uyoxy]

Hi team,

I'd love to take on the task of integrating NFT minting into the LandRegistry approval flow.

I’ll review the current logic and update the contract to automatically mint an NFT with relevant land metadata once approval occurs. I have experience with Solidity and ERC721 integration, ensuring smooth and secure on-chain workflows.

Looking forward to contributing!

[fishonamos]

Hi @SoarinSkySagar What is the update?