-
Notifications
You must be signed in to change notification settings - Fork 86
feat: add FreezeTransaction
#716
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
feat: add FreezeTransaction
#716
Conversation
|
awesome PR, thank you! Would it be possible to also add an example script thought (hardcoded to solo ofc? just for documentation purposes with a warning about required permissions. I'm fine with not adding integration tests however |
|
Won't there be a problem with the CI, because when it runs the example it could stop the network 😄 |
|
@Dosik13: that's a fair point. would need an unfreeze. ok, changelog rebase and we're good, thanks :D |
Signed-off-by: dosi <[email protected]>
Signed-off-by: dosi <[email protected]>
Signed-off-by: dosi <[email protected]>
Signed-off-by: dosi <[email protected]>
Signed-off-by: dosi <[email protected]>
Signed-off-by: dosi <[email protected]>
Signed-off-by: dosi <[email protected]>
948cb0f to
140dbb0
Compare
Description:
Implemented the
FreezeTransactionclass andFreezeTypeenum that allow freezing network.FreezeTypeenum class implementation with support for all freeze types (FREEZE_ONLY, PREPARE_UPGRADE, FREEZE_UPGRADE, FREEZE_ABORT, TELEMETRY_UPGRADE)FreezeTransactionclass implementation that enables freezing the network with configurable parameters (start_time, file_id, file_hash, freeze_type)FreezeTransactionclassFreezeTypeenum classFreezeTransactionandFreezeTypeto__init__.pyfile in__all__listRelated issue(s):
Fixes #260
Notes for reviewer:
Didn't add integration tests as it is hard and unnecessary to test. (I believe it won't be used by the users)
Checklist