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

pset: Add input explicit value, assets, and proofs, and issuance blinding flag #1157

Merged
merged 8 commits into from
Sep 6, 2022

Conversation

achow101
Copy link
Contributor

@achow101 achow101 commented Sep 1, 2022

Adds fields for explicit input value and assets, and their proofs. Also adds a flag to indicate whether an issuance should be blinded.

Closes #1153

doc/pset.mediawiki Outdated Show resolved Hide resolved
@apoelstra
Copy link
Member

Done reviewing 902b736:

Mostly just nits, except the nUtxo IsConfidential thing which needs to be fixed. This is a super well-organized PR, thank you!!

Copy link
Member

@apoelstra apoelstra left a comment

Choose a reason for hiding this comment

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

ACK 9ecbf38

@psgreco psgreco merged commit b4935a5 into ElementsProject:master Sep 6, 2022
gwillen added a commit to gwillen/elements that referenced this pull request Mar 2, 2023
apoelstra added a commit to ElementsProject/rust-elements that referenced this pull request Apr 23, 2024
eb85a23 ci: cargo update cc (Leonardo Comandini)
6320c7c pset: input: add explicit amount and asset, and their proofs (Leonardo Comandini)

Pull request description:

  They were added in ElementsProject/elements#1157

ACKs for top commit:
  apoelstra:
    ACK eb85a23

Tree-SHA512: f3126e66f210f0ee13f4d4751325f586ae8e35290791602a2066b3fe2b71914d51f2e012f20fed1ddd0690dc011586a8e1a6d115fc8815a959aff618f826486b
apoelstra added a commit to ElementsProject/rust-elements that referenced this pull request Apr 23, 2024
908cb5e pset: input: merge: add missing fields (Leonardo Comandini)
cd6ad48 pset: input: test issuance (Leonardo Comandini)
0453428 pset: input: handle blinded_issuance (Leonardo Comandini)
1bde7a1 pset: input: add blinded issuance flag (Leonardo Comandini)

Pull request description:

  Added in ElementsProject/elements#1157

ACKs for top commit:
  apoelstra:
    ACK 908cb5e this looks good for now. Would be good to actually implement the blinding of course

Tree-SHA512: 12039dcf0f68493cf959d84c64742b01367c847599c4960428f6813d9945933d526764e7787f8bb9836b34d217cb8a7d3003f38271ae79500dfc1b5477a720da
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 unblinded value and asset input fields to PSET
4 participants