-
Notifications
You must be signed in to change notification settings - Fork 24
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
feat(permissions): declare available permissions #232
Conversation
Merci beaucoup Pierre pour cette contribution Il faudra aussi reprendre la commande d'installation d'un sous-module pour pouvoir créer des droit à l'installation du sous-module et aussi gérer les permission sur les objects (+ les droit sur les objets SITE VISITES, etc...)
|
FROM | ||
( | ||
VALUES | ||
('MONITORINGS', 'ALL', 'C', False, 'Monitoring') |
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.
Il faudra utiliser des labels plus précis pour chaque permission.
Et en effet, dans Monitoring, il y a surement des permissions globales sur le module, mais aussi sur des objets plus spécifiques comme les sites, les visites...
834f43a
to
bfd5728
Compare
Cette PR est incomplète, car il y a d'autres niveaux de permissions dans Monitoring (sur les objets sites et visites, mais aussi pour que chaque sous-module puisse déclarer ses permissions disponibles). Mais à voir, par @joelclems. Voir ce qui est fait dans Modulator - https://github.com/PnX-SI/gn_modulator/blob/main/backend/gn_modulator/migrations/versions/b78eaab0a6e3_gn_modulator_permission_available.py Et PnX-SI/gn_modulator@fda2b80#diff-d14bffee01085f730a12e2ba596a0fe0cc9fcb05cb35b810540d0167a84320ff |
FROM | ||
( | ||
VALUES | ||
('MONITORINGS', 'ALL', 'R', False, 'Accéder à monitoring') |
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.
A renommer "Accéder au module", en cohérence avec Modulator
il faudrait aussi
|
Pour les sous-modules déjà installés, ça sera OK, car la migration des permissions quand on passe à GeoNature 2.13.0 se charge de les remettre à plat sur chaque module, sans héritage. |
No description provided.