Skip to content

Conversation

@nhussein11
Copy link
Collaborator

πŸ“ Description

Provide a clear and concise description of your changes.

πŸ” Review Preference

Choose one:

  • βœ… I have time to handle formatting/style feedback myself
  • ⚑ Docs team handles formatting (check "Allow edits from maintainers")

πŸ€– AI-Ready Docs

If content changed, regenerate AI files:

  • βœ… I ran python3 scripts/generate_llms.py
  • ⚑ Docs team will regenerate (check "Allow edits from maintainers")

βœ… Checklist

@nhussein11 nhussein11 self-assigned this Oct 23, 2025
Copilot AI review requested due to automatic review settings October 23, 2025 23:27
@nhussein11 nhussein11 requested a review from a team as a code owner October 23, 2025 23:27
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This pull request replaces the existing PolkaVM-focused NFT deployment guide with a comprehensive EVM-based guide that covers multiple deployment approaches. The new guide demonstrates how to deploy ERC-721 NFT contracts using standard EVM tooling including JavaScript/Ethers.js, Remix IDE, Hardhat, and Foundry.

Key changes:

  • Complete replacement of the PolkaVM-specific NFT deployment tutorial with an EVM-focused multi-approach guide
  • Addition of practical deployment examples for four different EVM development workflows
  • Updates to navigation and AI-generated documentation files to reflect the new content structure

Reviewed Changes

Copilot reviewed 30 out of 34 changed files in this pull request and generated no comments.

Show a summary per file
File Description
smart-contracts/cookbook/smart-contracts/deploy-nft.md Removed entire PolkaVM-based NFT deployment tutorial
smart-contracts/cookbook/smart-contracts/deploy-nft-evm.md Added comprehensive EVM-based NFT deployment guide covering Ethers.js, Remix, Hardhat, and Foundry
smart-contracts/cookbook/smart-contracts/.nav.yml Updated navigation to point to new EVM guide
llms.txt Updated documentation page count and added new guide to index
.ai/site-index.json Updated metadata timestamps and hashes for affected documentation
.ai/pages/*.md Removed code examples from multiple tutorial and reference pages (collapsible sections remain)

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@nhussein11 nhussein11 added B0 - Needs Review Pull request is ready for review C1 - Medium Medium priority task A0 - New Content Pull request contains new content pages labels Oct 24, 2025
@eshaben eshaben self-assigned this Oct 27, 2025
@eshaben eshaben requested a review from 0xlukem October 30, 2025 03:07
0xlukem

This comment was marked as outdated.

Copy link
Contributor

@0xlukem 0xlukem left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚒

@0xlukem 0xlukem removed the B0 - Needs Review Pull request is ready for review label Oct 30, 2025
@0xlukem 0xlukem added the B1 - Ready to Merge Pull request is ready to be merged label Oct 30, 2025
@eshaben eshaben merged commit 64e08a5 into staging/product-ia Oct 30, 2025
2 of 3 checks passed
@eshaben eshaben deleted the nhussein11/add-deploy-nft branch October 30, 2025 18:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A0 - New Content Pull request contains new content pages B1 - Ready to Merge Pull request is ready to be merged C1 - Medium Medium priority task

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants