ZEP-25 : Corrige la commande de migration + instructions MEP #3485
Conversation
| @@ -5,10 +5,10 @@ | |||
| from django.db import transaction | |||
| from django.shortcuts import get_object_or_404 | |||
| from django.utils.encoding import smart_text | |||
| from django.utils.text import slugify | |||
There was a problem hiding this comment.
@gustavi J'ai fais cette modification dans ta PR parce que flake8 n'était pas content. Tu me confirmes que c'était la bonne chose à faire ? J'importe from zds.utils import slugify un peu plus bas.
There was a problem hiding this comment.
si ça marche c'est good pour moi :)
There was a problem hiding this comment.
Dans la mesure où on utilise jamais from django.utils.text import slugify dans le projet, je suppose que oui. :)
There was a problem hiding this comment.
Et c'est toi qui a importé from zds.utils import slugify dans ta seconde version de ton script de migration mais tu avais laissé celui-ci aussi. Du coup, flake8 n'était pas content. J'ai choisi celui qu'on utilise partout entre les 2.
There was a problem hiding this comment.
pas de souci, j'ai codé ça rapidement et il était tard :)
QA
À tester en beta.
Corrigé :
Note @SpaceFox il faut réimporter la BDD de la beta car je l'ai un peu cassé niveau contenus/tags et surtout pour s'assurer que la commande est OK sur la prod.