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

Empêcher un non directeur d'éditer les décisions d'attribution #1214

Open
ac-dylan opened this issue Nov 14, 2024 · 1 comment · May be fixed by #1217
Open

Empêcher un non directeur d'éditer les décisions d'attribution #1214

ac-dylan opened this issue Nov 14, 2024 · 1 comment · May be fixed by #1217
Assignees

Comments

@ac-dylan
Copy link
Collaborator

Suite à deux tickets sur ce problème :
https://sesam.in.orion.education.fr/sesam/view.php?id=512025
https://sesam.in.orion.education.fr/sesam/view.php?id=511986
Des établissements ont généré des décisions d'attributions avec un autre nom que celui du directeur.

Solution :
Actuellement le bouton "Rééditer les décisions d'attribution" n'apparait seulement pour le directeur. (capture d'écran)
Sur ce même principe je pense qu'il faudrait que seul le directeur puisse effectuer la génération initiale des décisions d'attribution.

Solution alternative :
Le bouton "Rééditer les décisions d'attribution" ne semble pas changer le nom du CE.
Il faudrait que ce bouton demande de confirmer qu'on est CE avant de pouvoir soumettre ce formulaire et que cela remplace le nom renseigné dans les DA.

Cela éviterai que ce problème se reproduise.

PS : Vérifier que le problème n'est pas le même pour les Abrogations.

Image

@ac-dylan
Copy link
Collaborator Author

Voici la condition qui sert à définir le nom du Directeur au moment de la génération de la DA
director = establishment.confirmed_director || establishment.users.directors.first

confirmed_director remonte bien le nom du CE
users.directors.first remonte le nom qui a été utilisé par erreur dans le ticket

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

Successfully merging a pull request may close this issue.

3 participants