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

Add StakeValidatorHash to MustPayToOtherScript constraint #428

Merged
5 commits merged into from
Apr 22, 2022

Conversation

ghost
Copy link

@ghost ghost commented Apr 21, 2022

Should fix #424.


Pre-submit checklist:

  • Branch
    • Tests are provided (if possible)
    • Commit sequence broadly makes sense
    • Key commits have useful messages
    • Relevant tickets are mentioned in commit messages
    • Formatting, materialized Nix files, PNG optimization, etc. are updated
  • PR
    • Self-reviewed the diff
    • Useful pull request description
    • Reviewer requested

@ghost ghost marked this pull request as ready for review April 21, 2022 16:47
@ghost ghost requested review from sjoerdvisscher and koslambrou April 22, 2022 10:29
@ghost
Copy link
Author

ghost commented Apr 22, 2022

I think it would be nice to test it properly to make sure that it works but I have no idea how to do it yet.

@koslambrou
Copy link
Contributor

I think it would be nice to test it properly to make sure that it works but I have no idea how to do it yet.

We'll plan a testing strategy for the tx library when we rewrite it

@ghost ghost merged commit 1b8232d into main Apr 22, 2022
@ghost ghost deleted the must-pay-to-other-script-stake branch April 22, 2022 14:43
raduom pushed a commit to raduom/plutus-apps that referenced this pull request Apr 27, 2022
This pull request was closed.
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.

Extend TxConstraint's MustPayToOtherScript constructor to allow staked addresses
1 participant