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

FRAME - Feedback and local tweaks from UCB run-of-show incoperated #790

Open
1 of 9 tasks
Tracked by #708
nuke-web3 opened this issue Jul 27, 2023 · 0 comments
Open
1 of 9 tasks
Tracked by #708
Assignees

Comments

@nuke-web3
Copy link

nuke-web3 commented Jul 27, 2023

  • (Kian) Pallet Coupling: link back to this part of their rust exam. In fact, might be good to teach them supertraits in the rust exam as well.
  • Contribute diagrams for Hold and Freeze back to Substrate, akin to this.
  • They are asked to implement a pretty simple StorageValue and StorageMap here, would be good to back-reference.
  • Consider introducing dev_mode at the end, or as we go through, and how certain details are elided in dev_mode.
  • Some names for aggregate enums is outdated, should be RuntimeCall etc. Update Enum Names in Outer Enum Talk #803
  • In outer enums lecture, we cover what a cool thing type RuntimeEvet is, that you can access future types in abstract ways through traits. But we should also cover why not all pallets use them, and have to define their own RuntimeEvent, in order to add additional bounds. A similar example of this is that some pallets, despite having access to Currency or similar, need to define their own CurrencyBalance in order to fulfill types.
  • in DPoS assignment template, it would be good to provide further boilerplate about how to mock find_author.
  • In all assignment templates, it would be good to provide default and expected FreeId and HoldId.
  • Pallet instancing, if not clearly communicated, should be mentioned at a high level with am emphasis, because it comes relevant in XCM locations.
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

No branches or pull requests

4 participants