-
Notifications
You must be signed in to change notification settings - Fork 125
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-cli Plutus Datum JSON format support #357
Conversation
rust/src/utils.rs
Outdated
@@ -675,6 +675,7 @@ impl BigInt { | |||
return None; | |||
} | |||
match u64_digits.len() { | |||
0 => Some(to_bignum(0)), |
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.
unrelated to this PR, this was just a bug similar to the issue we had for serializing 0 as BigInt
before.
this commit is just for the detailed schema (but has some parts of the basic/no-schema format) as I couldn't find test cases. Further commits will be made once I can get some test cases. Based on: https://github.com/input-output-hk/cardano-node/blob/c1efb2f97134c0607c982246a36e3da7266ac194/cardano-api/src/Cardano/Api/ScriptData.hs#L254 + using `cardano-cli`.
be8542a
to
b20668b
Compare
@lisicky : review and resolve conflicts plz |
# Conflicts: # rust/src/utils.rs
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.
/check
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.
/check
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.
To reduce risk of conflicts, we need merge #345 before this PR
# Conflicts: # rust/src/utils.rs
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.
/check
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.
/check
# Conflicts: # rust/src/utils.rs
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.
/check
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.
/check
Implements both JSON formats used by cardano-node/cardano-cli
Based on: https://github.com/input-output-hk/cardano-node/blob/c1efb2f97134c0607c982246a36e3da7266ac194/cardano-api/src/Cardano/Api/ScriptData.hs#L254