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

Update nodejs client documentation + small fixes to JSON RPC client #610

Merged
merged 5 commits into from
Sep 30, 2022

Conversation

danimoh
Copy link
Member

@danimoh danimoh commented Jan 11, 2022

Pull request checklist

  • All tests pass. Demo project builds and runs.
  • I have resolved any merge conflicts.

What's in this pull request?

This PR updates the documentation for the nodejs client and the JSON RPC client.
The separate .md documentations as well as the usage instructions included in the clients and the sample configurations file have been updated.
All deprecated options are now reported as being deprecated.
Additionally, two small fixes for the reading of parameters in the JSON RPC client are included.

@danimoh danimoh requested a review from mar-v-in January 11, 2022 08:37
@danimoh danimoh self-assigned this Jan 11, 2022
Parameters of log action were not read correctly when a tag was passed as single
parameter. In this case the passed tag was interpreted as log level and the
default tag used.
Now calls like `node remote.js --silent mining` correctly treat the --silent as
boolean flag instead of assigning the following value to it. Previously, 'mining'
would have been assigned to `silent` and the action `mining` would have been
omitted.
@jeffesquivels jeffesquivels merged commit 393d7ca into master Sep 30, 2022
@jeffesquivels jeffesquivels deleted the daniel/update-documentation branch September 30, 2022 22:04
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.

3 participants