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
cardano-node: 1.19.1-patched -> 1.20.0 #2142
Conversation
e09eef7
to
113ee6f
Compare
223114d
to
323fb02
Compare
bors try |
tryBuild failed: |
e571aaf
to
8b9932e
Compare
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.
Seems like swagger<->json tests are failing
@@ -1398,18 +1398,59 @@ components: | |||
passphrase: *lenientPassphrase | |||
address_index: *addressIndex | |||
|
|||
TransactionMetadatum: &TransactionMetadatum | |||
TransactionMetadataValue: &TransactionMetadataValue |
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.
We need to confirm this with @AlexIOHK . In theory, 1.20.0 supports both approaches for metadata, the one we currently use with direct JSON conversion, and this one that is more of a JSON description of the metadata.
I'd have preferred going for the JSON description initially, but we had to get partners started with something and went for the direct JSON conversion which was ready. If we change that, it'll break existing integrations (although that is rather trivially fixed).
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.
OK I have changed the API to use the "no schema" mapping from cardano-api. I think it's the same as before. The PR for changing to the detailed schema is #2147.
8b9932e
to
2d41f15
Compare
e5ad680
to
b335bae
Compare
bors r+ |
2142: cardano-node: 1.19.1-patched -> 1.20.0 r=rvl a=rvl ### Issue Number ADP-307 / #2098 ### Overview Updates cardano-node to [1.20.0](https://github.com/input-output-hk/cardano-node/releases/tag/1.20.0) There are quite a few upstream API changes. Snapshot PR is input-output-hk/cardano-haskell#28. Co-authored-by: Rodney Lorrimar <rodney.lorrimar@iohk.io>
bors r- |
Canceled. |
bors r+ |
Build succeeded: |
2147: Use JSON "detailed schema" for API TxMetadata r=KtorZ a=rvl ### Issue Number ADP-307 / #2073 ### Overview Changes the API to use the "detailed" JSON schema for transaction metadata. With this scheme, API users will always see exactly the same JSON in the transaction history and cardano explorer as what they submitted with the transaction. ### Comments We may wish to support both formats in the API. This PR is based on the branch of #2142. Co-authored-by: Rodney Lorrimar <rodney.lorrimar@iohk.io>
Issue Number
ADP-307 / #2098
Overview
Updates cardano-node to 1.20.0
There are quite a few upstream API changes.
Snapshot PR is input-output-hk/cardano-haskell#28.