Thanks to visit codestin.com
Credit goes to github.com

Skip to content
This repository was archived by the owner on Jan 24, 2025. It is now read-only.

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 force-pushed the daniel/update-documentation branch from f5b4d4a to 393d7ca Compare September 30, 2022 21:53
@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 subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants