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
CU-1tf7kpr - Picasso v0 runtime - part 1 #478
Conversation
`Tokens` should never be used, except for `Assets` dependency The `Assets` pallet is acting as a router forwarding PICA operations to `Balance` and any other token operation to `Tokens`. By using `Tokens` instead of `Assets`, the pallet will not be able to operate on PICA.
Task linked: CU-1tf7kpr Picasso V0 runtime ugprade |
3f13311
to
0cb27ba
Compare
0cb27ba
to
b0ca667
Compare
b0ca667
to
f336612
Compare
Codecov Report
@@ Coverage Diff @@
## main #478 +/- ##
==========================================
+ Coverage 60.49% 60.75% +0.25%
==========================================
Files 141 141
Lines 20309 20312 +3
==========================================
+ Hits 12286 12340 +54
+ Misses 8023 7972 -51
Continue to review full report at Codecov.
|
Discussed offline to merge this, to allow for collaborations. |
This reverts commit be98233.
Factory: currency_factory::{Pallet, Storage, Event<T>} = 53, | ||
GovernanceRegistry: governance_registry::{Pallet, Call, Storage, Event<T>} = 54, | ||
Assets: assets::{Pallet, Call, Storage} = 55, | ||
CrowdloanRewards: crowdloan_rewards::{Pallet, Call, Storage, Event<T>} = 56, | ||
Vesting: vesting::{Call, Event<T>, Pallet, Storage} = 57, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@hussein-aitlahcen this pr doesn't add these pallets to the benchmark section
currency-factory = { package = "pallet-currency-factory", path = "../../frame/currency-factory", default-features = false } | ||
governance-registry = { package = "pallet-governance-registry", path = "../../frame/governance-registry", default-features = false } | ||
assets = { package = "pallet-assets", path = '../../frame/assets', default-features = false } | ||
crowdloan-rewards = { package = "pallet-crowdloan-rewards", path = '../../frame/crowdloan-rewards', default-features = false } | ||
vesting = { package = "pallet-vesting", path = "../../frame/vesting", default-features = false } | ||
bonded-finance = { package = "pallet-bonded-finance", path = "../../frame/bonded-finance", default-features = false } |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
missing std features as well as runtime-benchmark features
This PR introduce simple pallets for the V0.
I'll create a second, more specific one for Apollo.