chore: update polkadot dependencies #426
Conversation
tjwelde
force-pushed
the
tw-1647-polkadot0911
branch
from
4b78f82 to
5069bb2
Compare
| @@ -7,6 +7,9 @@ const chainProperties = TYPE_REGISTRY.createType('ChainProperties', { | |||
| ss58Format: 38, | |||
| }) | |||
| TYPE_REGISTRY.setChainProperties(chainProperties) | |||
| TYPE_REGISTRY.register({ | |||
There was a problem hiding this comment.
Q: Do we still need this? I don't see it changed for delegations or DIDs as well.
EDIT: I tried removing it and it did not break the integration tests.
EDIT 2: Nevermind, it broke the unit tests 😂
| @@ -342,6 +342,23 @@ describe('handling queries to data not on chain', () => { | |||
| }) | |||
| }) | |||
|
|
|||
| describe.only('hierarchyDetails', () => { | |||
There was a problem hiding this comment.
describe('hierarchyDetails', () => {
EDIT: I also tried removing .only and of course all the rest of the delegation integration tests were also passing, so this can be removed safely.
|
I’ve noticed that during upgrade many versions lost the |
|
@arty-name yes, that is correct. I tried with |
fixes KILTProtocol/ticket#1647
This PR updates the polkadot dependencies to work with the new 0.9.11 polkadot update.
I tried it out with a sample application, which connects to either peregrine stg, or spiritnet and I try to read a balance.
It works for both networks.
The type registry could be theoretically removed, but for downwards compatibility, we leave it in for the next release.
How to test:
Checklist: