Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update learn-nft-pallets.md. #6539

Merged
merged 7 commits into from
Feb 18, 2025
Merged

Update learn-nft-pallets.md. #6539

merged 7 commits into from
Feb 18, 2025

Conversation

FFilipUnique
Copy link
Contributor

Added an introductory paragraph mentioning all currently available NFT pallets. This update is part of the Polkadot NFTMozaic deliverables mandate. This is the initial commit.

Added an introductory paragraph mentioning all currently available NFT pallets. This update is part of the Polkadot NFTMozaic deliverables mandate. This is the initial commit.
Copy link
Contributor

@filippoweb3 filippoweb3 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 for this addition. In general, the added content is well-structured. Please note the following remarks when pushing future PRs to the Wiki:

  • keep the content unbiased (the current PR is biased towards Uniques)
  • avoid making recommendations about which pallet to use unless specifically asked
  • when citing projects try to find if there is a section in the Wiki to link to, if not, use the main website of the project
  • when mentioning things like "it is the most used" it is better to link to a proof to back up the statement

I hope this helps :)

docs/learn/learn-nft-pallets.md Outdated Show resolved Hide resolved
docs/learn/learn-nft-pallets.md Outdated Show resolved Hide resolved
docs/learn/learn-nft-pallets.md Outdated Show resolved Hide resolved
docs/learn/learn-nft-pallets.md Outdated Show resolved Hide resolved
docs/learn/learn-nft-pallets.md Outdated Show resolved Hide resolved
docs/learn/learn-nft-pallets.md Outdated Show resolved Hide resolved
@filippoweb3 filippoweb3 added A3 - Changes Requested Pull Request needs to be updated. A0 - Do Not Merge Pull request should not yet be merged. labels Feb 11, 2025
@filippoweb3 filippoweb3 removed the A3 - Changes Requested Pull Request needs to be updated. label Feb 17, 2025
@filippoweb3 filippoweb3 merged commit 19f30fb into w3f:master Feb 18, 2025
1 check passed
@filippoweb3 filippoweb3 removed the A0 - Do Not Merge Pull request should not yet be merged. label Feb 18, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants