colonyNetwork
colonyNetwork copied to clipboard
Improve reputation miner user experience
This PR does the following:
-
Arguments to the reputation miner client executable are simplified and described better. Removed:
network
,localProviderAddress
,localPort
,providerAddress
, all in favor ofrpcEndpoint
-
Almost all arguments got sensible defaults. Now only
privateKey
orminerAddress
have to be specified. -
All arguments have corresponding environment variables with the
REP_MINER
prefix (e.g.privateKey
isREP_MINER_PRIVATE_KEY
) -
The docker image got adjusted to that (and also should yield a more consistent build)
-
The whole configuration is shown when the reputation miner is started
-
All docs are adjusted to the new arguments and workflow
-
In the docs, the possibility to stake for reputation mining was added (directly from a form in the docs)
-
Alex fixed two issues that were related to first time miners.
The commit I've added pages the previous mining cycles on syncing. This is exactly the same issue that we were facing in the frontend in a few different areas that Raul fixed. The same solution is done here - instead of asking for all events in one big request, we split it up in to multiple requests and join all the responses together.