-
Notifications
You must be signed in to change notification settings - Fork 292
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
Conversation
There was a problem hiding this 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.
dcrjson/walletsvrcmds.go
Outdated
@@ -318,6 +318,14 @@ func NewGetBalanceCmd(account *string, minConf *int) *GetBalanceCmd { | |||
} | |||
} | |||
|
|||
type GetContractHashCmd struct { |
There was a problem hiding this comment.
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.
dcrjson/walletsvrcmds.go
Outdated
FilePath []string | ||
} | ||
|
||
func NewGetContractHashCmd(filepaths []string) *GetContractHashCmd { |
There was a problem hiding this comment.
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 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
NewGetContractHashCmd
dcrjson/walletsvrresults.go
Outdated
@@ -41,6 +41,11 @@ type GetBestBlockResult struct { | |||
Height int64 `json:"height"` | |||
} | |||
|
|||
// GetContractHash models the data from the getcontracthash command. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
GetContractHashResult
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry about that Dave.
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.