feat(Cantines): nouveau champ siren_unite_legale pour commencer à gérer les cantines sans SIRET propre #5081
Conversation
raphodn
force-pushed
the
raphodn/backend-canteen-siren_unite_legale
branch
from
61d0ae2 to
16bf35a
Compare
data/models/canteen.py
Outdated
| self.check_siret_rules() | ||
| super(Canteen, self).save(force_insert, force_update, using, update_fields) | ||
|
|
||
| def check_siret_rules(self): |
There was a problem hiding this comment.
C'est plutôt le check dans l'autre sens qui me semble utile : si pas de SIRET alors SIREN obligatoire
A priori, ça ne pose pas de pbm d'avoir SIREN et SIRET (mais on en veut pas forcément non plus)
There was a problem hiding this comment.
le hic c'est que ca rajoute un nouveau check assez fort - l'obligation d'avoir un SIRET - qui n'est pas respecté par une ~40 aine de cantines en DB. est-ce qu'on veut rajouter cette règle ici et maintenant ? (mes premiers commit allaient dans ce sens, puis j'ai revert)
There was a problem hiding this comment.
A priori, ça ne pose pas de pbm d'avoir SIREN et SIRET (mais on en veut pas forcément non plus)
ah ok !
There was a problem hiding this comment.
j'ai enlevé les règles du coup ici : 535db2c
There was a problem hiding this comment.
et j'ai ouvert une PR séparée - #5088 - pour rendre le SIRET obligatoire, mais ca peut venir après, une fois qu'on est sûr des impacts et tout et tout 😅
raphodn
force-pushed
the
raphodn/backend-canteen-siren_unite_legale
branch
from
16bf35a to
fd53cd8
Compare
Modifications apportées :
siren_unite_legale