Viewers & edit collaborators on a project can't see the list of collaborators
Les mainteneurs répondent en général sous 1 jour
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 3/5
- Temps estimé
- 1-2 jours
- Accessibilité débutants
- 67/100
Piste de recherche
Start with the collaboration view in documentcloud/projects/views.py around line 675, then read the permission definitions in documentcloud/projects/rules.py around lines 47 and 53. Trace how the project collaborator list is gated and identify a collaborator-level check for viewing it while keeping the existing permission for settings and invitations. Done means viewers and editors can see the collaborator list, while the existing admin-only actions remain gated.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
If projects are intended to be a collaborative space, this is a confusing user experience. If someone adds me to a project as a viewer or editor, I have no idea who added me at this point.
The collaboration view set is gated by an if here:
https://github.com/MuckRock/documentcloud/blob/d5f6a31764fafd315603503c6c75dc5b4e1c6726/documentcloud/projects/views.py#L675
projects.change_project_all is the permission which gates it.
This is defined here:
https://github.com/MuckRock/documentcloud/blob/d5f6a31764fafd315603503c6c75dc5b4e1c6726/documentcloud/projects/rules.py#L53
as anyone who holds can_change
which is defined here:
https://github.com/MuckRock/documentcloud/blob/d5f6a31764fafd315603503c6c75dc5b4e1c6726/documentcloud/projects/rules.py#L47
That means only authenticated admins of the project can see the collaborator list.
So the collaborators list is tied to the same permission as editing project settings and inviting people. That’s why only admins see it. The fix is to give the list its own collaborator-level check, and leave this permission as is I think.
- Langage dominant
- Python
- Étoiles
- 55
- Forks
- 10
- Merge moyen
- 1 j 13 h
- PR mergées (30 j)
- 22
Préparer son environnement
Ce projet ne fournit ni conteneur de développement, ni Dockerfile, ni guide de contribution : l'installation est à votre charge. Commencez par son README, et consultez notre guide de la première contribution pour les étapes générales.
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de MuckRock/documentcloud
-
Remove cpuprofile dependencyOuverte
Difficulté 1/5 Moins d'une heure Accessibilité débutants 62/100
MuckRock/documentcloud#537 · 1 commentaire ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 3/5 1-2 jours Accessibilité débutants 50/100
MuckRock/documentcloud#539 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 3/5 1-2 jours Accessibilité débutants 66/100
MuckRock/documentcloud#531 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 4/5 3-5 jours Accessibilité débutants 55/100
MuckRock/documentcloud#521 ·
Les mainteneurs répondent en général sous 1 jour
-
processing
Difficulté 4/5 3-5 jours Accessibilité débutants 48/100
MuckRock/documentcloud#509 ·
Les mainteneurs répondent en général sous 1 jour
Toutes les issues de MuckRock/documentcloud
Issues similaires
-
Difficulté 2/5 1-3 heures Accessibilité débutants 83/100
Les mainteneurs répondent en général sous 1 jour
-
comp/desktop P3 type/feature
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
NousResearch/hermes-agent#154285 ·
Les mainteneurs répondent en général sous 1 jour
-
docs: qualify copied and custom kubectl debug policy notesPeut-être pris @nym01 l’a pris aujourd’hui. Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
kubescape/cel-admission-library#166 · 1 commentaire ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 68/100
SACGF/variantgrid#2804 · 1 commentaire ·
Les mainteneurs répondent en général sous 1 jour
-
[BUG] HTTP lifecycle dispatch treats a NO_PROXY-only environment as corporate egress and skips the resolve-and-pinned sessionPeut-être pris @bodapatisaikrishna l’a pris aujourd’hui. Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 76/100
Les mainteneurs répondent en général sous 1 jour