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

Wire up required signers to build and build-raw cli commands #3123

Merged
merged 1 commit into from
Aug 26, 2021

Conversation

Jimbo4350
Copy link
Contributor

@Jimbo4350 Jimbo4350 commented Aug 25, 2021

Resolves #3106

@catch-21
Copy link
Contributor

Confirmed this works, passes my script validation when checking for tx signatories.
Also confirmed the negative case where I try to submit tx without required signature, throws on submit: (MissingRequiredSigners (fromList [KeyHash "1ed5f183a76122092ac0f7f23427ca22385d5d6e8166300eb71a8b28"]))

Copy link
Contributor

@nc6 nc6 left a comment

Choose a reason for hiding this comment

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

Looks good to me

@nc6
Copy link
Contributor

nc6 commented Aug 26, 2021

bors merge

@iohk-bors
Copy link
Contributor

iohk-bors bot commented Aug 26, 2021

Build succeeded:

@iohk-bors iohk-bors bot merged commit a586a48 into master Aug 26, 2021
@iohk-bors iohk-bors bot deleted the jordan/plumb-required-signers-cli branch August 26, 2021 08:28
iohk-bors bot added a commit that referenced this pull request Sep 15, 2021
3127: Script context equivalence shell scripts r=Jimbo4350 a=Jimbo4350

This PR implements two Plutus scripts that check the equivalence of the Plutus `ScriptContext` with a custom redeemer built from the accompanying transaction. The custom redeemer mimics the `ScriptContext` data type. Depends on #3122 and #3123

Co-authored-by: Jordan Millar <jordan.millar@iohk.io>
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.

[FR] - Cli to pass required signer (reqSignerHashes) to plutus script for txInfoSignatories
3 participants