-
Notifications
You must be signed in to change notification settings - Fork 45
Detail Solid GitHub permissions associated to roles #78
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
Conversation
Solution associated to conversation on solid-contrib/information#48
|
@Mitzi-Laszlo Went through and applied updates as requested. You can see the differences in this commit. A few details on the rationale behind some of the changes:
|
RubenVerborgh
left a comment
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.
Agree in general, just think there is a confusion wrt Editor.
|
|
||
| The Solid Manager, who is also an editor, is appointed by the Solid Director. The Solid Manager is responsible for formalizing the outcome of any votes. | ||
|
|
||
| Editors belong to the [Editors team](https://github.com/orgs/solid/teams/editors) in the [Solid GitHub Organization](https://github.com/solid) and have [_Maintain Permissions_](https://help.github.com/en/articles/repository-permission-levels-for-an-organization#permission-levels-for-repositories-owned-by-an-organization) on all repositories therein. |
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 think the Editor role is being misunderstood (in general, not just in this PR).
To me (and to a typical W3C group), an editor is always in relation to a document. So you are an editor of document X or Y; you're not an editor in absolute terms.
The role of "editor" seems to have gotten some special meaning that IMHO is not needed or desired. Editors just need write access to the solid/specification repository, and the authority to approve/merge pull requests to a specific document.
|
This pull request has been succeeded by #95. This branch and its associated version history was merged into that branch and then modified further. Closing this pull request. |
Solution associated to conversation on solid-contrib/information#48