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

Add txCollateralRequirement to TransactionCtx. #2956

Merged
merged 2 commits into from
Oct 7, 2021
Merged

Add txCollateralRequirement to TransactionCtx. #2956

merged 2 commits into from
Oct 7, 2021

Conversation

jonathanknowles
Copy link
Contributor

@jonathanknowles jonathanknowles commented Oct 7, 2021 •
edited
Loading

Issue Number

ADP-1042

Summary

This PR:

  • adds field txCollateralRequirement to TransactionCtx.
  • uses the txCollateralRequirement within Wallet.selectAssets to populate the SelectionParams record.

@jonathanknowles jonathanknowles marked this pull request as ready for review October 7, 2021 09:27
@jonathanknowles jonathanknowles force-pushed the jonathan-samuel/adp-1042 branch from f6b4f62 to c4029aa Compare October 7, 2021 09:50
@jonathanknowles
Copy link
Contributor Author

bors r+

@jonathanknowles jonathanknowles mentioned this pull request Oct 7, 2021
Closed
@iohk-bors
Copy link
Contributor

iohk-bors bot commented Oct 7, 2021

Build succeeded:

@iohk-bors iohk-bors bot merged commit b1ded06 into master Oct 7, 2021
@iohk-bors iohk-bors bot deleted the jonathan-samuel/adp-1042 branch October 7, 2021 11:00
KtorZ added a commit that referenced this pull request Oct 7, 2021
@KtorZ
TEMPORARY HACK: use inputs as collateral. Remove after #2954/#2956
9dfc725
@KtorZ KtorZ mentioned this pull request Oct 7, 2021
Merged
KtorZ added a commit that referenced this pull request Oct 7, 2021
@KtorZ
TEMPORARY HACK: use inputs as collateral. Remove after #2954/#2956
e810190
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sevanspowell sevanspowell sevanspowell approved these changes

Assignees

@jonathanknowles jonathanknowles

@sevanspowell sevanspowell

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@jonathanknowles @sevanspowell