Skip to content
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.

Sign up for GitHub

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

Jump to bottom

test collateral inputs param on transaction view cli command #948

Merged
merged 1 commit into from
Mar 8, 2022
Merged

test collateral inputs param on transaction view cli command #948

merged 1 commit into from
Mar 8, 2022

Conversation

saratomaz
Copy link
Collaborator

No description provided.

@saratomaz saratomaz closed this Mar 2, 2022
@saratomaz saratomaz reopened this Mar 2, 2022
@saratomaz saratomaz force-pushed the tx_collateral_inputs branch 7 times, most recently from bbf5501 to 11ff61f Compare March 3, 2022 12:03
@saratomaz saratomaz marked this pull request as ready for review March 3, 2022 13:55
@saratomaz saratomaz requested a review from mkoura March 3, 2022 13:59
@saratomaz saratomaz linked an issue Mar 3, 2022 that may be closed by this pull request
transaction view --> "Add collateral inputs friendly output" #942
Closed
8 tasks
mkoura
mkoura requested changes Mar 7, 2022
View reviewed changes
cardano_node_tests/utils/tx_view.py Outdated Show resolved Hide resolved
cardano_node_tests/utils/tx_view.py Outdated Show resolved Hide resolved
cardano_node_tests/utils/tx_view.py Outdated Show resolved Hide resolved
@saratomaz saratomaz force-pushed the tx_collateral_inputs branch 4 times, most recently from a3a45d8 to bb7dcfc Compare March 8, 2022 14:38
@saratomaz saratomaz requested a review from mkoura March 8, 2022 14:38
mkoura
mkoura requested changes Mar 8, 2022
View reviewed changes
Copy link
Collaborator

@mkoura mkoura left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

One last thing, otherwise 👍

cardano_node_tests/utils/tx_view.py Outdated Show resolved Hide resolved
@saratomaz saratomaz requested a review from mkoura March 8, 2022 15:19
mkoura
mkoura approved these changes Mar 8, 2022
View reviewed changes
Copy link
Collaborator

@mkoura mkoura left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@mkoura mkoura merged commit f8db27a into IntersectMBO:master Mar 8, 2022
@saratomaz saratomaz deleted the tx_collateral_inputs branch August 25, 2022 15:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mkoura mkoura mkoura approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

transaction view --> "Add collateral inputs friendly output"
2 participants
@saratomaz @mkoura