Skip to content
This repository has been archived by the owner on Nov 6, 2020. It is now read-only.

JSON-RPC interfaces with documentation #4276

Merged
merged 7 commits into from
Jan 24, 2017
Merged

JSON-RPC interfaces with documentation #4276

merged 7 commits into from
Jan 24, 2017

Conversation

maciejhirsz
Copy link
Contributor

Related to #2646 and #1905. Contains #4275 which should be reviewed first.

Adding missing methods, descriptions and examples to the JS interfaces, from which the new wiki JSON-RPC documentation has been generated and updated.

@maciejhirsz maciejhirsz added A0-pleasereview 🤓 Pull request needs code review. F5-documentation 📑 Documentation needs fixing, improving or augmenting. M6-rpcapi 📣 RPC API. labels Jan 23, 2017
@@ -156,11 +172,21 @@ export default class Parity {
.execute('parity_killAccount', inAddress(account), password);
}

listAccounts (count, offset = null, blockNumber = 'latest') {
return this._transport
.execute('parity_listAccounts', count, inAddress(offset), inBlockNumber(blockNumber));
Copy link
Contributor

@jacogr jacogr Jan 24, 2017

Choose a reason for hiding this comment

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

We probably need to push the result through some formatter - I'm assuming it will contain addresses. (If the output is the same as eth_accounts could just be a map over outAddress)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Array of addresses or null (parity RPC wiki is up to date now). Will fix.

@maciejhirsz maciejhirsz removed the F5-documentation 📑 Documentation needs fixing, improving or augmenting. label Jan 24, 2017
@jacogr jacogr added A8-looksgood 🦄 Pull request is reviewed well. and removed A0-pleasereview 🤓 Pull request needs code review. labels Jan 24, 2017
@gavofyork gavofyork merged commit a58fad0 into master Jan 24, 2017
@gavofyork gavofyork deleted the mh-rpcsync branch January 24, 2017 21:02
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A8-looksgood 🦄 Pull request is reviewed well. M6-rpcapi 📣 RPC API.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants