-
Notifications
You must be signed in to change notification settings - Fork 102
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
[IMPORT][ALEMBIC]fix:downgrade revision add constraint error #2938
base: feat/import
Are you sure you want to change the base?
[IMPORT][ALEMBIC]fix:downgrade revision add constraint error #2938
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## feat/import #2938 +/- ##
===============================================
+ Coverage 78.44% 81.80% +3.36%
===============================================
Files 89 120 +31
Lines 7213 9300 +2087
===============================================
+ Hits 5658 7608 +1950
- Misses 1555 1692 +137
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
58fdc22
to
fd291fc
Compare
22e3600
to
3cc485f
Compare
Cannot add constraint unicity_t_mappings_fields_name_field (fix by delete duplicate vaulue) Cannot add foreign key id_theme to bib_field (fix by set default value ) Reviewed-by: andriacap
272d87f
to
fa626ee
Compare
Rebasé, mais reste encore à relire. |
dcdde5a
to
d64cff9
Compare
On a discuté de la stratégie sur les downgrades, et ceux-cis ne doivent pas supprimer de donner pour si jamais ils sont lancé en prod. C’est à l’admin / aux devs de supprimer manuellement les données qui posent problème (après s’être assuré de ne pas supprimer de donner essentiel). On est donc plutôt d’avis de ne pas intégrer cette évolution. |
Cannot add constraint unicity_t_mappings_fields_name_field (fix by delete duplicate vaulue)
Cannot add foreign key id_theme to bib_field (fix by set default value )
Maybe this PR should be gathered with other works about the topic :
update/downgrade import
Reviewed-by: andriacap