-
Notifications
You must be signed in to change notification settings - Fork 10
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 docs #168
Update docs #168
Conversation
This is actually an important distinction, maybe it could be rewritten better though. Zaino is taking over responsibility for serving all "non-miner" clients from zcashd, with this change, zebra is now only responsible for serving zaino or serving "miner clients" direct. So we should keep the distinction in the docs. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
distinction between miner and non-miner clients should be kept as is important use case detail.
I understand that miner and non-miner are important distinctions.. my question was about changing all instances of the the word "miner" to "validator". "miner" makes more sense in the context of To put it another way, eventually there won't be any miners, but zaino will still be serving validators (once CrossLink lands). |
Ahh sorry I understand now. Yes I am with this change. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me, Im not sure if this will conflict with the name change PR though?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I am not sure about the term validator and am concerned that it's ambiguous.
I'm unclear on whether validator can now be used in place of miner, but I noticed that validator is used earlier in the doc, and then "miner" shows up later. Maybe we can just switch to "validator" in all cases?