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

dcrjson: Add pay to contract hash support. #1260

Merged
merged 1 commit into from
Oct 8, 2018

Conversation

githubsands
Copy link
Contributor

@githubsands githubsands commented Jun 4, 2018

Required changes to implement BIP175 decred/dcrwallet#1132 into Decred. Allows two new RPC methods (1) getcontracthashes and (2) getpaytocontractaddress. This loose coupling allows users to grab just the contract(s) hash if needed and a contract P2PKH.

@davecgh davecgh changed the title dcrjson: additions for dcrwallet#1132 dcrjson: Add pay to contract hash support. Jun 4, 2018
Copy link
Member

@davecgh davecgh left a comment

Choose a reason for hiding this comment

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

Commit titles and descriptions must not refer to github PR or issue numbers.

Only the PR description itself can refer to the PR or issue number.

@@ -318,6 +318,14 @@ func NewGetBalanceCmd(account *string, minConf *int) *GetBalanceCmd {
}
}

type GetContractHashCmd struct {
Copy link
Member

Choose a reason for hiding this comment

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

Missing comment for auto doc generation.

FilePath []string
}

func NewGetContractHashCmd(filepaths []string) *GetContractHashCmd {
Copy link
Member

Choose a reason for hiding this comment

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

Missing comment for auto doc generation.

FilePath []string
}

// NewGetBalanceCmd returns a new instance which can be used to issue a
Copy link
Member

Choose a reason for hiding this comment

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

NewGetContractHashCmd

dcrjson/walletsvrcmds.go Show resolved Hide resolved
@@ -41,6 +41,11 @@ type GetBestBlockResult struct {
Height int64 `json:"height"`
}

// GetContractHash models the data from the getcontracthash command.
Copy link
Member

Choose a reason for hiding this comment

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

GetContractHashResult

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Sorry about that Dave.

@davecgh davecgh merged commit 52109ea into decred:master Oct 8, 2018
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.

2 participants