-
Notifications
You must be signed in to change notification settings - Fork 4
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
permettre les champs vide dans l'admin du model Programme #1562
Conversation
56ddf31
to
376c6bf
Compare
☂️ Python Coverage
Overall Coverage
New FilesNo new covered files... Modified Files
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
J'ai du diviser la migration car elle ne se terminait jamais
|
||
class Migration(migrations.Migration): | ||
|
||
atomic = False |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Supression de la transction pour gagner en perf
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Je suis en train d'exécuter les migrations en local FYI
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@etchegom : C'est Ok pour toi pour les migrations ?
] | ||
|
||
operations = [ | ||
migrations.RunSQL( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
J'ai extrait la mise à jour de la valeur par défaut pour les perf
) | ||
uuid = models.UUIDField(default=uuid.uuid4, editable=False) | ||
nom = models.CharField(max_length=255) | ||
numero_operation = models.CharField(max_length=255, null=True) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Suppression de null=True sur tout les champs de type texte suite à la recommandation de SonarCloud
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Quality Gate passedIssues Measures |
Je me suis retrouvé couincé quand j'ai tenté de corriger un programme :
Cette PR ne corrige que l'admin de l'app programme, il y a surment d'autres modeles à modifier