-
Notifications
You must be signed in to change notification settings - Fork 23
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
Fix generated textEnvelope type on registering a Stake Address Registration Certificate #390
Conversation
919681c
to
f41ac38
Compare
LGTM (nice usage of Relevant doc to write the tests: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 🎉. Can you squash your commits?
9b7a015
to
0913bd2
Compare
did a test with cli8.16.0.1 by manually changing the textEnvelope type to
so it looks like the cli is not aware of this type name yet for processing a deregistration certificate. and i guess its the same for the registration certificate aswell. |
Golden tests done in cli: IntersectMBO/cardano-cli#499 |
c336d3c
to
206f22a
Compare
206f22a
to
ef62e85
Compare
…ano-api-8.27.0.0 Upgrade to `cardano-api-8.27.0.0`
Changelog
Context
Fixes the issue #465 in cardano-cli
How to trust this PR
Highlight important bits of the PR that will make the review faster. If there are commands the reviewer can run to observe the new behavior, describe them.
Checklist