Closed Neopallium closed 1 year ago
Merging #1426 (24c6a2a) into develop (3cb3786) will increase coverage by
0.07%
. The diff coverage is86.66%
.:exclamation: Current head 24c6a2a differs from pull request most recent head 75b0514. Consider uploading reports for the commit 75b0514 to get more accurate results
@@ Coverage Diff @@
## develop #1426 +/- ##
===========================================
+ Coverage 53.88% 53.95% +0.07%
===========================================
Files 146 146
Lines 31884 31934 +50
===========================================
+ Hits 17180 17231 +51
+ Misses 14704 14703 -1
Impacted Files | Coverage Δ | |
---|---|---|
pallets/runtime/common/src/fee_details.rs | 58.02% <0.00%> (-1.47%) |
:arrow_down: |
src/chain_spec.rs | 0.00% <ø> (ø) |
|
pallets/sudo/src/lib.rs | 27.15% <45.45%> (+2.67%) |
:arrow_up: |
primitives/src/identity.rs | 84.61% <75.00%> (-5.39%) |
:arrow_down: |
primitives/src/identity_id.rs | 79.47% <84.61%> (+1.14%) |
:arrow_up: |
pallets/bridge/src/lib.rs | 85.42% <90.90%> (+0.25%) |
:arrow_up: |
pallets/bridge/src/genesis.rs | 90.62% <100.00%> (+0.79%) |
:arrow_up: |
pallets/identity/src/keys.rs | 77.27% <100.00%> (+0.34%) |
:arrow_up: |
pallets/identity/src/lib.rs | 23.72% <100.00%> (+0.25%) |
:arrow_up: |
pallets/identity/src/types.rs | 18.51% <100.00%> (+10.18%) |
:arrow_up: |
... and 6 more |
changelog
new features
Default
trait fromAccountId
.modified API
Controller
andAdmin
changed toOption<AccountId>
.Key
changed toOption<AccoundId>
.Deposits
changed toOption<DepositInfo<T::AccountId>>
.Sudo.KeyChanged
changed toOption<AccountId>
.modified logic
InvalidAccountKey
error will be thrown. Only Asset identities have no primary key.