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 #3503 : plus possible de creer un tag vide #3515

Merged
merged 1 commit into from
Apr 9, 2016
Merged

Fix #3503 : plus possible de creer un tag vide #3515

merged 1 commit into from
Apr 9, 2016

Conversation

gustavi
Copy link
Contributor

@gustavi gustavi commented Apr 6, 2016

Q R
Type de modification correction de bug
Ticket(s) (issue(s)) concerné(s) #3503

QA

  • Essayer de créer un tag vide (en laissant une virgule à la fin) sur un contenu

@gustavi gustavi added S-BUG Corrige un problème C-Back Concerne le back-end Django labels Apr 6, 2016
@coveralls
Copy link

Coverage Status

Coverage remained the same at 87.309% when pulling 9ab0765 on gustavi:fix-3503 into 60e65f4 on zestedesavoir:release-v18.

@Emeric54
Copy link
Contributor

Emeric54 commented Apr 6, 2016

QA:

En entrant simplement une , dans la liste des tags , je peux encore créer un tag vide.

@sandhose
Copy link
Contributor

sandhose commented Apr 9, 2016

Euh, QA OK ?

Je viens de tester, et j'ai pas réussi à créer de tag vide... @Emeric54 ?

@Emeric54
Copy link
Contributor

Emeric54 commented Apr 9, 2016

Sur le coup, en regardant le code, je ne comprenais pas pourquoi on pouvait encore créer un tag vide... J'ai trouvé maintenant : je n'avais pas effectué les migrations nécessaires :-°

Je suis désolé, merci @sandhose d'être repassé par là. C'est donc tout bon ici.

@sandhose
Copy link
Contributor

sandhose commented Apr 9, 2016

OK, je merge

@sandhose sandhose merged commit be9492f into zestedesavoir:release-v18 Apr 9, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-Back Concerne le back-end Django S-BUG Corrige un problème
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants