Skip to content

Conversation

@UtkarshBhardwaj007
Copy link
Collaborator

@UtkarshBhardwaj007 UtkarshBhardwaj007 commented Nov 12, 2025

πŸ“ Description

Updated to use latest hardhat version + EVM backend + latest nodes release

πŸ” 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

Closes #1228

@UtkarshBhardwaj007 UtkarshBhardwaj007 requested a review from a team as a code owner November 12, 2025 23:23
@UtkarshBhardwaj007 UtkarshBhardwaj007 added B0 - Needs Review Pull request is ready for review C2 - High High priority task A2 - Maintenance Minor Pull request contains minor updates to an existing page (i.e., modifying parameters, steps, etc.) B1 - Ready to Merge Pull request is ready to be merged labels Nov 12, 2025
@eshaben eshaben requested a review from a team November 13, 2025 05:21
@eshaben eshaben removed the B1 - Ready to Merge Pull request is ready to be merged label Nov 13, 2025
Copy link
Collaborator

@nhussein11 nhussein11 left a comment

Choose a reason for hiding this comment

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

LGTM! thanks @UtkarshBhardwaj007 !

One minor thing, in line #36, we should change from pointing to https://faucet.polkadot.io/?parachain=1111 to just https://faucet.polkadot.io/, right? So people then select AssetHub manually, since the rpc we are showing is https://testnet-passet-hub-eth-rpc.polkadot.io

@UtkarshBhardwaj007
Copy link
Collaborator Author

UtkarshBhardwaj007 commented Nov 13, 2025

LGTM! thanks @UtkarshBhardwaj007 !

One minor thing, in line #36, we should change from pointing to https://faucet.polkadot.io/?parachain=1111 to just https://faucet.polkadot.io/, right? So people then select AssetHub manually, since the rpc we are showing is https://testnet-passet-hub-eth-rpc.polkadot.io

Thanks @nhussein11 , I think it's better to directly link to the faucet for Passet Hub, otherwise we would need to document this and ask people to select Passet Hub.
For now, we are still using Passet Hub and not Paseo Asset Hub RPC.

@nhussein11
Copy link
Collaborator

LGTM! thanks @UtkarshBhardwaj007 !
One minor thing, in line #36, we should change from pointing to https://faucet.polkadot.io/?parachain=1111 to just https://faucet.polkadot.io/, right? So people then select AssetHub manually, since the rpc we are showing is https://testnet-passet-hub-eth-rpc.polkadot.io

Thanks @nhussein11 , I think it's better to directly link to the faucet for Passet Hub, otherwise we would need to document this and ask people to select Passet Hub. For now, we are still using Passet Hub and not Paseo Asset Hub RPC.

Makes sense then, successfully tested all the commands. Many thanks! πŸš€

This now is ready for the @polkadot-developers/docs-formatting-team

@nhussein11 nhussein11 self-requested a review November 13, 2025 12:31
@nhussein11 nhussein11 mentioned this pull request Nov 13, 2025
6 tasks
Copy link
Collaborator

@dawnkelly09 dawnkelly09 left a comment

Choose a reason for hiding this comment

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

A few suggestions after running through Grammarly. Thank you for your contribution!

UtkarshBhardwaj007 and others added 5 commits November 13, 2025 16:25
Co-authored-by: Dawn Kelly <83190195+dawnkelly09@users.noreply.github.com>
Co-authored-by: Dawn Kelly <83190195+dawnkelly09@users.noreply.github.com>
Co-authored-by: Dawn Kelly <83190195+dawnkelly09@users.noreply.github.com>
Co-authored-by: Dawn Kelly <83190195+dawnkelly09@users.noreply.github.com>
Co-authored-by: Dawn Kelly <83190195+dawnkelly09@users.noreply.github.com>
@UtkarshBhardwaj007
Copy link
Collaborator Author

A few suggestions after running through Grammarly. Thank you for your contribution!

Thanks @dawnkelly09 , accepted all suggestions. Could you please review again and merge please? Hoping to get this in before tomorrow as sub0 starts tomorrow.

Copy link
Collaborator

@dawnkelly09 dawnkelly09 left a comment

Choose a reason for hiding this comment

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

🚒

@dawnkelly09 dawnkelly09 added B1 - Ready to Merge Pull request is ready to be merged and removed B0 - Needs Review Pull request is ready for review labels Nov 13, 2025
@UtkarshBhardwaj007 UtkarshBhardwaj007 merged commit 991cf5f into polkadot-developers:master Nov 13, 2025
11 of 12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A2 - Maintenance Minor Pull request contains minor updates to an existing page (i.e., modifying parameters, steps, etc.) B1 - Ready to Merge Pull request is ready to be merged C2 - High High priority task

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Update Hardhat documentation for the web0 hackathon

4 participants