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

fix: bug sur les gestionnaires sur Admin #1638

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

Guilouf
Copy link
Collaborator

@Guilouf Guilouf commented Jan 16, 2025

Quoi ?

Bug rapporté par Pauline, trace ici

Pourquoi ?

Problème de contrainte d'intégrité sur les gestionnaires, alors qu'aucune n'est définie dans le modèle. Le problème se situe ici: /~https://github.com/gip-inclusion/le-marche/pull/109/files#diff-5e753eeddbaaab6c4cf67e55983d1712f6442f0d7c910907e9bd86d1eeb61f71R58

Du SQL a été exécuté, sans que le modèle de données ne reflète ces changements.

Comment ?

Une nouvelle migration à été créée pour remettre à plat le modèle sans contraintes fantômes.

Commentaire

Plus de contraintes fantômes, mais plus de contrainte tout court, un même utilisateur peut être gestionnaire 2 fois pour une même structure. Faut il ajouter une contrainte sur le modèle intermédiaire SiaeUser ?

@Guilouf Guilouf added the bug Something isn't working label Jan 16, 2025
@Guilouf Guilouf self-assigned this Jan 16, 2025
@Guilouf Guilouf marked this pull request as ready for review January 17, 2025 10:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant