Skip to content
This repository has been archived by the owner on Jun 15, 2021. It is now read-only.

Produce 'Roles' doc #46

Merged
merged 38 commits into from
Jul 10, 2018
Merged

Produce 'Roles' doc #46

merged 38 commits into from
Jul 10, 2018

Conversation

cbeams
Copy link
Contributor

@cbeams cbeams commented Jun 29, 2018

Similar to the doc we already have on the Proposals infrastructure and process, this doc on Roles will capture everything about Roles infrastructure and process under the Bisq DAO.

It will:

@cbeams cbeams changed the title Produce 'Roles' doc [WIP] Produce 'Roles' doc Jun 29, 2018
cbeams added 14 commits June 29, 2018 11:13
These TODOs are captured as checklist items in the pull request
description.
 - Use fewer words (this has nothing specific to do with Roles)
 - Rework Roles section to use Duties, Rights and Owners language
 - Reference new Roles doc where appropriate with footnotes
Following suit with recent, similar changes to the Proposal doc's Roles
section. Structure, footnotes, etc are the same.
This is a full pass over the doc with changes to most sections.
Several sections remain incomplete.
@cbeams cbeams changed the title [WIP] Produce 'Roles' doc Produce 'Roles' doc Jul 10, 2018
@cbeams cbeams merged commit 2fc32d7 into bisq-network:master Jul 10, 2018
cbeams added a commit that referenced this pull request Jul 10, 2018
@cbeams
Copy link
Contributor Author

cbeams commented Jul 10, 2018

@bisq-network/role-owners, I have merged this new 'Roles' documentation without putting it through review because I will shortly put together two proposals that reference this doc, and those proposals will be an opportunity to provide feedback on the most important parts of it.

Of course, if you'd like to read and review the document in a standalone fashion, please do. Just add an issue or submit a pull request for any issues you encounter as you read through it. Also feel free to post questions and suggestions here on this pull request, even now that it is merged and closed. I'll still respond to them.

You can find the live version of the doc at https://docs.bisq.network/roles.html.

/cc @m52go

@m52go
Copy link
Contributor

m52go commented Jul 13, 2018

@cbeams just read through this, great doc.

Only 1 suggestion: consider putting the 'Processes' section before 'The Roles Maintainer role' section. The preceding sections discuss elements concerning all roles, the 'Role Maintainer' section only covers that one role, and then 'Processes' goes back to discussing elements that concern all roles.

I see why you did it this way, but switching the two would flow a bit better, I think.

cbeams added a commit to cbeams/bisq-docs that referenced this pull request Jul 17, 2018
@cbeams
Copy link
Contributor Author

cbeams commented Jul 17, 2018

@m52go wrote:

1 suggestion: consider putting the 'Processes' section before 'The Roles Maintainer role' section.

Good call, I've just done this in #77. Thanks!

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.

2 participants