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.

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

Unresolved TxOut pretty printing bug resolution #387

Merged
merged 5 commits into from
Mar 21, 2024
Merged

Conversation

mmontin
Copy link
Collaborator

@mmontin mmontin commented Mar 20, 2024

This fixes #385.
The issue was in function lookupOutput where TxOuts would only be resolved when they had a datum which was a mistake.
This also fixes a small redundancy when printing reference scripts.

@mmontin mmontin requested a review from florentc March 20, 2024 20:32
@mmontin mmontin changed the title Txoutref printing bug resolution TxOut printing bug resolution Mar 20, 2024
@mmontin mmontin changed the title TxOut printing bug resolution Unresolved TxOut pretty printing bug resolution Mar 20, 2024
Copy link
Member

@florentc florentc left a comment

Choose a reason for hiding this comment

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

Reviewed and improved during meeting.

@mmontin mmontin merged commit fbfa4e9 into main Mar 21, 2024
6 checks passed
@mmontin mmontin deleted the pretty-txoutref branch March 21, 2024 15:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Some transaction inputs are not resolved when pretty-printing
2 participants