-
Notifications
You must be signed in to change notification settings - Fork 117
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
Generalise multiverse method names to support all proofs. #496
Conversation
There are more changes to make along the same lines, but we might as well get these in. |
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 🎉
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.
Good to keep names in sync 👍🏽
I think this might need a rebase to get the CI/merge queue passing. |
ae4ce24
to
682f905
Compare
682f905
to
e7beb2f
Compare
Not sure why, but the postgres itest keeps failing consistently... |
This PR renames multiverse store and archive method names to reflect the fact that they are now handle transfer proofs in addition to issuance proofs.
We should merge #473 first.