Skip to content

Support supplying contract hash to issueasset RPC - #1010

Merged
apoelstra merged 1 commit into
ElementsProject:masterfrom
stevenroose:fw-issue-asset-contract-hash
Sep 16, 2021
Merged

Support supplying contract hash to issueasset RPC#1010
apoelstra merged 1 commit into
ElementsProject:masterfrom
stevenroose:fw-issue-asset-contract-hash

Conversation

@stevenroose

@stevenroose stevenroose commented Jun 15, 2021

Copy link
Copy Markdown
Contributor

forward port of #993.
Closes #985.

@stevenroose
stevenroose force-pushed the fw-issue-asset-contract-hash branch from d856472 to b8deedc Compare July 28, 2021 14:32
@apoelstra

Copy link
Copy Markdown
Member

Can you rebase this and see if that fixes CI

@stevenroose
stevenroose force-pushed the fw-issue-asset-contract-hash branch from b8deedc to 2bcaeb2 Compare September 8, 2021 13:08
@stevenroose

Copy link
Copy Markdown
Contributor Author

Done

@apoelstra apoelstra added this to the 0.21 milestone Sep 8, 2021
@apoelstra

Copy link
Copy Markdown
Member

Looks like you added an option to the RPC help but didn't add it again to the list of RPCs at the bottom of wallet/rpcwallet.cpp.

This redundancy will be removed in 22.0 but in 0.21 we just enforce consistency in CI.

@stevenroose
stevenroose force-pushed the fw-issue-asset-contract-hash branch from 2bcaeb2 to 47d43f1 Compare September 15, 2021 17:23
@stevenroose

Copy link
Copy Markdown
Contributor Author

I added that, sorry

@stevenroose

Copy link
Copy Markdown
Contributor Author

Somehow CI doesn't seem to run on this one.. Does anyone understand why that is?

@apoelstra

Copy link
Copy Markdown
Member

It looks like it ran and all the checks passed.

@apoelstra apoelstra left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

ACK 47d43f1

@apoelstra
apoelstra merged commit 388d47f into ElementsProject:master Sep 16, 2021
gwillen pushed a commit that referenced this pull request Jun 1, 2022
Basically just reconstructed the diff since so much code had been moved.
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 contract_hash to issueasset

2 participants