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

[FR] - Build and build-raw to include interface for Voting and Proposal script (guardrails-script) to be used as reference script. #809

Closed
CarlosLopezDeLara opened this issue Jul 2, 2024 · 3 comments

Comments

@CarlosLopezDeLara
Copy link
Contributor

CarlosLopezDeLara commented Jul 2, 2024

Internal/External
Internal if an IOHK staff member.

Area
Conway - Guardrails script PV3

Describe the feature you'd like
Right now the only way to use the voting and proposal script (guardrails script) is by using the script file directly with --proposal-script-file , it would be very useful to have options like --proposal-tx-in-reference and --vote-tx-in-reference

Might be good idea to rename the flag to guardrail-script or constitution-script , so it would be

--guardrails-script-file
--guardrails-tx-in-reference 
...

OR

--constitution-script-file
--constitution-tx-in-reference 
...

Additional context / screenshots
The guardrails script is available here: https://github.com/IntersectMBO/constitution-priv

Still a private repo, so copying the description below:

Being a smart contract, the constitution script is a validator function of the form:

const_script :: BuiltinData -> BuiltinUnit
The sole argument to this function is the BuiltinData-encoded V3.ScriptContext. Note the absence of the 2 extra arguments, previously known as Datum argument and the Redeemer argument. Since V3 and CIP-69, the Datum and Redeemer values are not passed anymore as separate function arguments, but embedded inside the V3.ScriptContext argument. The "proposal under investigation" is also embedded inside the ScriptContext.

Datum is not provided for the "constitution script", since it is not a spending validator. Redeemer will be provided to the "constitution script", but the current script implementations ignore any value given to it (see Clause D).

@CarlosLopezDeLara CarlosLopezDeLara changed the title [FR] - Build and build raw to include interface for Proposal script (guardrails-script) to be used as reference script. [FR] - Build and build-raw to include interface for Proposal script (guardrails-script) to be used as reference script. Jul 2, 2024
@CarlosLopezDeLara CarlosLopezDeLara changed the title [FR] - Build and build-raw to include interface for Proposal script (guardrails-script) to be used as reference script. [FR] - Build and build-raw to include interface for Voting and Proposal script (guardrails-script) to be used as reference script. Jul 2, 2024
@Jimbo4350
Copy link
Contributor

#822

Copy link

github-actions bot commented Aug 4, 2024

This issue is stale because it has been open 30 days with no activity. Remove stale label or comment or this will be closed in 120 days.

@github-actions github-actions bot added the Stale label Aug 4, 2024
@CarlosLopezDeLara
Copy link
Contributor Author

Resolved by #822

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants