colonyNetwork icon indicating copy to clipboard operation
colonyNetwork copied to clipboard

Improve reputation miner user experience

Open chmanie opened this issue 1 year ago • 1 comments

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 of rpcEndpoint

  • Almost all arguments got sensible defaults. Now only privateKey or minerAddress have to be specified.

  • All arguments have corresponding environment variables with the REP_MINER prefix (e.g. privateKey is REP_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.

chmanie avatar Aug 30 '23 14:08 chmanie

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.

area avatar Sep 13 '23 15:09 area