-
Notifications
You must be signed in to change notification settings - Fork 14
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
Remove the custom migration logic from all runtimes #454
Comments
@pendulum-chain/product this is more of a chore but nice-to-have and low effort. We can clean up some unnecessary code and repositories with this. |
Hey team! Please add your planning poker estimate with Zenhub @b-yap @bogdanS98 @ebma @gianfra-t @TorstenStueber |
I think we can reuse this ticket and remove the custom migration logic after the 1.1.0 upgrade is completed. Otherwise we would need to re-add them soon. |
Right, let's do that. I added the ticket as a blocker to the Zenhub dependencies. |
@ebma is this possible to be worked on now, or only after the upgrade on Pendulum? |
@vadaynujra only once we have updated the runtimes we can remove this extra code. We left this ticket also as a reminder to remove the extra logic after the runtime upgrade that bumps to In any case, this is not blocking anything. |
As part of our efforts to upgrade the dependencies to v0.9.42, we added custom migration logic to all runtimes. Now that these changes are rolled out and all the migrations applied, we can remove that extra logic again.
TODO
CustomOnRumtimeUpgrade
implementations againThe text was updated successfully, but these errors were encountered: