Skip to content

Use the term "shadow" and store all names and hashes. - #152

Merged
tweag-ev-ak merged 1 commit into
mainfrom
shadow-hashes
Mar 16, 2026
Merged

tweag-ev-ak merged 1 commit into
mainfrom
shadow-hashes

Conversation

@tweag-ev-ak

Copy link
Copy Markdown
Collaborator

closes #133

@tweag-ev-ak
tweag-ev-ak marked this pull request as draft February 26, 2026 06:28
@tweag-ev-ak
tweag-ev-ak marked this pull request as ready for review March 9, 2026 10:59

@axman6 axman6 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Better late than never...

Comment thread docs/specification.md
scripts:
- script_hash: 921169a988ba72ffd6e9c269cadb3b53b5f360ff99f112d9b2ee30c4d74ad88b
- hash: 921169a988ba72ffd6e9c269cadb3b53b5f360ff99f112d9b2ee30c4d74ad88b
cbor_hex: 73475cb40a568e8da8a045ced110137e159f890ac4da883b6b17dc651b3a804973475cb40a568e8da8a045ced110137e159f890ac4da883b6b17dc651b3a804973475cb40a568e8da8a045ced110137e159f890ac4da883b6b17dc651b3a804973475cb40a568e8da8a045ced110137e159f890ac4da883b6b17dc651b3a8049

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I found the other day that this format no longer works, it now looks something like:

...
    substitutions:
      - name: "[WRT] WingRiders Governance Token"
        hash: "c0ee29a85b13209423b10447d3c2e6a50641a15c57770e27cb9d5073"
        source:
          cbor_hex:
            type: PlutusScriptV1
            description: "[WRT] WingRiders Governance Token"
            cborHex:  "590a080100...

The use of both cborHex and cbor_hex is unfortunate. Not an issue from this PR, but something that happened with the reimplementation of the ConfigMap. Ping @adithyaov

@tweag-ev-ak
tweag-ev-ak merged commit 715a964 into main Mar 16, 2026
2 checks passed
@tweag-ev-ak
tweag-ev-ak deleted the shadow-hashes branch March 16, 2026 12:23
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.

Add the substitutedScriptHash in the ExecutionContext

2 participants