Skip to content
This repository has been archived by the owner on Nov 15, 2023. It is now read-only.

Fix Statemint #885

Open
bkchr opened this issue Jan 4, 2022 · 4 comments
Open

Fix Statemint #885

bkchr opened this issue Jan 4, 2022 · 4 comments
Labels
I7-refactor Code needs refactoring. T7-system_parachains This PR/Issue is related to System Parachains.

Comments

@bkchr
Copy link
Member

bkchr commented Jan 4, 2022

We needed to fix Statemint, because at the upgrade from Shell to Statemint there was used the wrong crypto. We fixed it using this hacky pr: #867

A proper solution would now be to support running Statemint with different kind of cryptos, aka make the current version work with ed25519 and then switch over to sr25519 using a runtime upgrade.

@hbulgarini hbulgarini added the T7-system_parachains This PR/Issue is related to System Parachains. label Jan 19, 2022
@hbulgarini hbulgarini linked a pull request Jan 20, 2022 that will close this issue
@gilescope gilescope added this to the statemint-v7.0.0 milestone Jan 20, 2022
@hbulgarini hbulgarini removed this from the statemint-v7.0.0 milestone Jan 24, 2022
@gilescope
Copy link
Contributor

So what's left here is switching over statemint back to sr25519. Is it just setting SR versions of the initial_authorities that we need to do in the migration? (or will this have affected user key derivation on the front end?)

@bkchr
Copy link
Member Author

bkchr commented May 24, 2022

What front end? We will need to port over the authorities and do some client side changes.

@gilescope
Copy link
Contributor

Ok what client side changes do we need to make? can you point me at an example line of code of what your thinking about?

@bkchr
Copy link
Member Author

bkchr commented May 25, 2022

There is no example code for this. We will need to be able to import blocks using sr25519 and ed25519.

@the-right-joyce the-right-joyce added I7-refactor Code needs refactoring. and removed F6-refactor 🧹 labels Aug 12, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
I7-refactor Code needs refactoring. T7-system_parachains This PR/Issue is related to System Parachains.
Projects
None yet
Development

No branches or pull requests

4 participants