Skip to content

Conversation

@kapetan3sid
Copy link
Collaborator

This PR introduces two new pages in the Technical Reference section:

  • Assets – Explains how to work with assets on Asset Hub, including native assets, foreign assets, and asset management using the Assets pallet.
  • Smart Contracts – Describes smart contract deployment options on Asset Hub and outlines the technical differences between PolkaVM and REVM.

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 PR splits the existing "Assets and Smart Contracts" page into two dedicated pages—Assets and Smart Contracts—in the Technical Reference section. The refactoring improves content organization by separating asset management concepts from smart contract deployment options on Asset Hub.

Key changes:

  • Creates dedicated assets.md page covering Asset Hub's asset management features, pallets, and XCM integration
  • Introduces smart-contracts.md page explaining PolkaVM and REVM contract deployment options
  • Updates navigation files to reflect the new structure

Reviewed Changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 5 comments.

Show a summary per file
File Description
reference/polkadot-hub/smart-contracts.md New page documenting smart contract options (PolkaVM and REVM) on Asset Hub
reference/polkadot-hub/assets.md New page covering asset management, pallets, foreign assets, and XCM monitoring
reference/polkadot-hub/assets-and-smart-contracts.md Removed original combined page
reference/polkadot-hub/.nav.yml Updated navigation to link to separate assets and smart-contracts pages
.nav.yml Added Technical Reference section to main navigation

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

kapetan3sid and others added 5 commits October 21, 2025 15:38
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
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.

Thank you @kapetan3sid ! Left some comments, maybe after the refactoring of the assets page, the combination assets.md + smart-contracts.md is not that lenghty and it could fit everything under the same page

@eshaben
Copy link
Collaborator

eshaben commented Oct 23, 2025

@kapetan3sid let me know when it's ready. Seems like there are some major changes that need to be made and approved by Nico before I step in.

kapetan3sid and others added 3 commits October 27, 2025 12:46
Co-authored-by: Nicolás Hussein <80422357+nhussein11@users.noreply.github.com>
Co-authored-by: Nicolás Hussein <80422357+nhussein11@users.noreply.github.com>
@kapetan3sid
Copy link
Collaborator Author

kapetan3sid commented Oct 27, 2025

@nhussein11 I applied changes - this is ready for re-review

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.

Thank you very much @kapetan3sid !

Left some comments, but I believe those can be taken over by the @polkadot-developers/docs-formatting-team. Just a quick note, the ai files were not pushed and there are some instances of target={_blank} that I did not flag

kapetan3sid and others added 5 commits October 27, 2025 16:01
Co-authored-by: Nicolás Hussein <80422357+nhussein11@users.noreply.github.com>
Co-authored-by: Nicolás Hussein <80422357+nhussein11@users.noreply.github.com>
Co-authored-by: Nicolás Hussein <80422357+nhussein11@users.noreply.github.com>
Co-authored-by: Nicolás Hussein <80422357+nhussein11@users.noreply.github.com>
Co-authored-by: Nicolás Hussein <80422357+nhussein11@users.noreply.github.com>
@kapetan3sid
Copy link
Collaborator Author

@erin ready for your review

kapetan3sid and others added 3 commits October 28, 2025 13:54
@eshaben
Copy link
Collaborator

eshaben commented Oct 29, 2025

Assets Page:

I removed a lot of the technical references from this page, because they were used for like querying the data, but not most of the core extrinsics. Since this page is conceptual, I didn't think we need to callout the exact function signature and details. So I removed that.

I also removed the part about monitoring XCM transfers because it just felt out of place. I think this content should be in the XCM guides instead if needed.


Contracts Page:

Overall, this page doesn't seem to have a clear purpose.

It mentioned PolkaVM so I commented that out. Then what was left was very little and is duplicated elsewhere in the docs. Maybe someone can tell me what the purpose of this page is? Especially versus this page: https://beta-docs.polkadot.com/smart-contracts/for-eth-devs/contract-deployment/#revm-deployment

@brunopgalvao
Copy link
Contributor

Contracts Page:

Overall, this page doesn't seem to have a clear purpose.

It mentioned PolkaVM so I commented that out. Then what was left was very little and is duplicated elsewhere in the docs. Maybe someone can tell me what the purpose of this page is? Especially versus this page: https://beta-docs.polkadot.com/smart-contracts/for-eth-devs/contract-deployment/#revm-deployment

I agree. Also the Assets page should be less of a reference to the assets pallet API and more of how assets work on the Polkadot Hub. We need to be telling the story of what Polkadot Hub is, in terms of asset creation and management, smart contract functionality, etc.

kapetan3sid and others added 7 commits October 29, 2025 14:14
Co-authored-by: Bruno Galvao <brunopgalvao@gmail.com>
Co-authored-by: Bruno Galvao <brunopgalvao@gmail.com>
Co-authored-by: Bruno Galvao <brunopgalvao@gmail.com>
Co-authored-by: Bruno Galvao <brunopgalvao@gmail.com>
Co-authored-by: Bruno Galvao <brunopgalvao@gmail.com>
Co-authored-by: Bruno Galvao <brunopgalvao@gmail.com>
Co-authored-by: Bruno Galvao <brunopgalvao@gmail.com>
@eshaben
Copy link
Collaborator

eshaben commented Oct 29, 2025

Also the Assets page should be less of a reference to the assets pallet API and more of how assets work on the Polkadot Hub. We need to be telling the story of what Polkadot Hub is, in terms of asset creation and management, smart contract functionality, etc.

Yeah, I agree. So it sounds like this page will need DevRel attention then. So I propose we ship it with what's there and then a ticket be created for the backlog so that you or Nico can circle back to it.

@brunopgalvao
Copy link
Contributor

Also the Assets page should be less of a reference to the assets pallet API and more of how assets work on the Polkadot Hub. We need to be telling the story of what Polkadot Hub is, in terms of asset creation and management, smart contract functionality, etc.

Yeah, I agree. So it sounds like this page will need DevRel attention then. So I propose we ship it with what's there and then a ticket be created for the backlog so that you or Nico can circle back to it.

Sounds good to me.

@eshaben
Copy link
Collaborator

eshaben commented Nov 4, 2025

@brunopgalvao ok, I made sure your comments were addressed so this should be good to go for now. Then, whenever time permits, either you or Nico can circle back to it and make sure the content aligns with the goals of the page.

@eshaben eshaben requested a review from dawnkelly09 November 4, 2025 21:39
@kapetan3sid kapetan3sid marked this pull request as draft November 5, 2025 13:08
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.

🚢

@kapetan3sid kapetan3sid added the A0 - New Content Pull request contains new content pages label Nov 11, 2025
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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants