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

Supprimer la table AgentTerritorialAccessRight #4455

Open
3 tasks
Holist opened this issue Jul 19, 2024 · 0 comments
Open
3 tasks

Supprimer la table AgentTerritorialAccessRight #4455

Holist opened this issue Jul 19, 2024 · 0 comments
Milestone

Comments

@Holist
Copy link
Contributor

Holist commented Jul 19, 2024

Une fois les tâches réalisées l'option allow_to_manage_access_rights n'est plus vraiment utile et peut devenir allow_to_define_territory_admin et sert uniquement comme autorisation d'escalade de privilége pour certains agents déjà admin de territoire.
A voir techniquement comment faire au mieux mais on ne devrai pas avoir besoin de 2 tables différentes pour définir un admin de territoire et les droits lui permettant d'en définir d'autres. Cela porte à confusion.
On pourrait par exemple fusionner les tables AgentTerritorialAccessRight (qui ne contient plus que allow_to_define_territory_admin et la table AgentTerritorialRole qui est une relation entre agent et territoire et dont la seule utilité est de définir si un agent est admin de territoire.

@Holist Holist added this to the Espace Admin milestone Jul 19, 2024
@Holist Holist changed the title Rendre les policies cohérentes entre les admins de territoires et les droits de l'application Supprimer la table AgentTerritorialAccessRight Jul 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: 🔖 Ready
Development

No branches or pull requests

1 participant