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

chore: new pallet #32

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from
Draft

chore: new pallet #32

wants to merge 2 commits into from

Conversation

Daanvdplas
Copy link
Collaborator

This functionality already exists and is great for devs to kickstart the development. It creates a pallets folder with a fresh pallet folder ready to build.

Showing that it exists can also get early user feedback.

@Daanvdplas Daanvdplas requested a review from brunopgalvao July 17, 2024 12:33
@brunopgalvao
Copy link
Contributor

I like the idea! I would add a phrase before that to download/install Pop CLI (with a link)

Also, the running your parachain link in the README should be:

@Daanvdplas
Copy link
Collaborator Author

The link for "Learn about how to build using this template" was the best tutorial I could find for someone that needs info about how to start building their first pallet. I think we want something like that on the pop website, maybe trying to find the kitties example and update it.

@AlexD10S
Copy link
Collaborator

I'd prefer to point a link to our documentation instead of: https://subdaily.io/dev/learn-to-build-substrate-pallets-from-scratch-dissecting-the-frame-pallet/ This way, we can maintain it.

The line: 👷 Start with building a new pallet is a great idea. However, as with the previous point, can we create a tutorial in our documentation and link to it instead of adding basic code? This approach will be more consistent with the rest of the points.

pop new pallet <pallet_name>

@brunopgalvao
Copy link
Contributor

brunopgalvao commented Jul 24, 2024

Okay. We need to define exactly what we want to add to the Pop Docs. Do we want a section called "Add a pallet to a parachain" that highlights the pop new pallet <pallet_name> command? Or do we want something more comprehensive?

https://subdaily.io/dev/learn-to-build-substrate-pallets-from-scratch-dissecting-the-frame-pallet/

  • Is teaching about Substrate and pallets. Do we want to teach about Substrate and pallets in Pop Docs?

@brunopgalvao
Copy link
Contributor

@Daanvdplas
Copy link
Collaborator Author

Daanvdplas commented Sep 18, 2024

@brunopgalvao can you decide whether to use this or otherwise close this draft please

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.

3 participants