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 migrations des modèles IcaEffetsDeclarés, IcaPopulationCible, IcaPopulationRisque #1572

Closed
wants to merge 1 commit into from

Conversation

pletelli
Copy link
Collaborator

@pletelli pletelli commented Jan 31, 2025

Dans cette PR, le 4b84856 n'avait pas été suivi d'une migration. La voici.

J'ai une petite incertitude sur ce qu'il se passe dans ce cas où une migration est supprimée et une autre remplacée alors qu'elles ont déjà été jouées en prod.
Mais lorsque que je veux simplement en recrééer une, django ne détecte aucun changement.

Je pense qu'il faudrait avant de merger cette PR, faire en prod, staging et demo sur un python manage.py migrate data 0117

@pletelli
Copy link
Collaborator Author

Dans cette PR, le 4b84856 n'avait pas été suivi d'une migration. La voici.

J'ai une petite incertitude sur ce qu'il se passe dans ce cas où une migration est supprimée et une autre remplacée alors qu'elles ont déjà été jouées en prod. Mais lorsque que je veux simplement en recrééer une, django ne détecte aucun changement.

Je pense qu'il faudrait avant de merger cette PR, faire en prod, staging et demo sur un python manage.py migrate data 0117

En fait, comme ce sont des modèles managed=False, ça ne change absolument rien que le champ vrsdcl_ident soit défini avec primary_key=True même s'il ne l'est pas en réalité.
Par contre si il n'est pas défini comme ça, Django attend une colonne id, qui n'existe pas dans les données de base et ça fait le bazar. Je vais donc revenir à la situation initiale en ajoutant un commentaire d'explication.

@pletelli pletelli closed this Jan 31, 2025
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 this pull request may close these issues.

1 participant