Skip to content
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

Permissions au niveau des objets des modules monitoring #249

Closed
amandine-sahl opened this issue Sep 29, 2023 · 8 comments
Closed

Permissions au niveau des objets des modules monitoring #249

amandine-sahl opened this issue Sep 29, 2023 · 8 comments

Comments

@amandine-sahl
Copy link
Contributor

amandine-sahl commented Sep 29, 2023

Pour éviter toute confusion (et ne pas réimplémenter un pseudo héritage entre les permissions), nous souhaiterions que les droits des utilisateurs pour un type d'objet correspondent aux permissions définis pour ce type et non pour l'objet ALL.

Par exemple un module qui a des sites, des visites et des observations aurait les permissions disponibles suivantes :

  • ALL - R : accès au module
  • ALL - U : modification des paramètres du module
  • ALL - E : exporter les données du module
  • GNM_SITES - CRUD
  • GNM_VISITS - CRUD
  • GNM_OBSERVATIONS - CRUD
@TheoLechemia
Copy link
Member

MONITORING_ plutôt que GNM_ ?

@camillemonchicourt
Copy link
Member

Oui bien vu.
Je crois que c'est ce qui est fait dans Occtax etc, donc privilégier l'homogénéité.

@amandine-sahl
Copy link
Contributor Author

Après discussion les permissions seraient :

  • MONITORINGS_MODULES - R : accès au module
  • MONITORINGS_MODULES - U : modification des paramètres du module
  • MONITORINGS_MODULES - E : exporter les données du module
  • MONITORINGS_SITES - CRUD
  • MONITORINGS_GRP_SITES - CRUD
  • MONITORINGS_VISITS - CRUD (qui inclus les permissions des objets visites, observations et observations détails)

@MathRdt
Copy link

MathRdt commented Oct 20, 2023

Hello !
Merci pour cette amélioration des permissions sur le module monitoring !
Je voulais savoir si vous aviez une idée de la date de release de cette amélioration ?
C'est une feature qui nous intéresse beacuoup côté CREA avec @ColinVanReeth et on a hâte de pouvoir la tester !
Y a t'il des points bloquants ou des actions que je peux prendre pour valider cette feature ?

Bonne journée,
Mathieu

@amandine-sahl
Copy link
Contributor Author

Nous sommes en train de tester s'il n'y a pas des effets de bord non anticipés. Si vous voulez faire avancer la PR, n'hésitez pas à la tester tous les retours sont bon.

@MathRdt
Copy link

MathRdt commented Nov 7, 2023

Ok super, merci pour les informations.
Je vais tenter de l'implémenter sur notre environnement de dev, je vous fait des retours si j'ai des soucis ou comportements bizarre.

@MathRdt
Copy link

MathRdt commented Dec 5, 2023

Hello,
désolé pour le retard, j'ai été mobilisé sur d'autres sujets.
J'ai pu déployer cette version de dev et enfin la tester ! Pour la partie CRUD sur les différents modules tout semble bien se passer de mon côté, donc super !

Par contre le filtre sur l'appartenance des données ne semble pas fonctionner. Si je rajoute le filtre "Mes données", j'ai quand même le droit de voir tous les sites, visites, etc... Même ceux créés par une autre personne

@amandine-sahl
Copy link
Contributor Author

merci pour les tests.

Effectivement la portée n'est pas encore fonctionnelle, elle viendra dans un second temps.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants