-
Notifications
You must be signed in to change notification settings - Fork 22
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
🎁 [Feature Request]: Pallet Abstractions to Improve DX #257
Comments
Check out some of the ideas that @kianenigma has been working on https://github.com/kianenigma/flite In this example (at the time of writing), he has been able to create a wrapper around The example he gives is having Ideally, we could expand this idea to also include within a single pallet things like Balances, and also a concrete balance type. At that point, building a new pallet with Flite becomes trivial like a smart contract You can see that flite has some pretty non-trivial code / tricks inside of it, but the end pallet developed using it is super simple. |
Update on this: paritytech/polkadot-sdk#5584 |
Draft design seems to be: https://github.com/OpenZeppelin/polkadot-runtime-wrappers/tree/amar-oz-system-wrapper-poc |
So I see that this is done, but in ways different from what I have proposed here. Can we discuss why? Note that I don't think there is something very wrong with what is currently done, but it lacks a few features:
|
Thank you for the feedback! This was indeed our first iteration, and we intentionally kept it minimal to get something functional in place. We are absolutely open to suggestions for improving the macros and share your goal of making them more robust. We’ve already created follow-up issues to address the improvements, especially around type safety and better leveraging Rust’s type system. Our plan is to tackle these issues over the coming months to enhance the macros significantly. Let us know if there are specific adjustments you’d recommend as we iterate. |
@kianenigma appreciate the feedback. As @4meta5 said we know this first iteration is far from perfect, but we hope it still is a significant improvement fro DevEx and simplifies the parachain runtime code. |
templates
Macros PR: OpenZeppelin/openzeppelin-pallet-abstractions#1
Templates PR: #300
Contribution Guidelines
The text was updated successfully, but these errors were encountered: