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.

Sign up for GitHub

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

Jump to bottom

Fix #3626 : corrige les espaces dans l'ajout de tags sur les contenus #3627

Merged
merged 1 commit into from
May 26, 2016
Merged

Fix #3626 : corrige les espaces dans l'ajout de tags sur les contenus #3627

merged 1 commit into from
May 26, 2016

Conversation

gustavi
Copy link
Contributor

@gustavi gustavi commented May 26, 2016

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

À QA LE PLUS RAPIDEMENT POSSIBLE

QA

  • ajouter à un contenu les tags « foo, bar » (l'espace est importante).
  • Vérifier que le tag créé ne comprend pas d'espace et ne renvoie pas de 404

@coveralls
Copy link

coveralls commented May 26, 2016 •
edited
Loading

Coverage Status

Coverage remained the same at 87.402% when pulling 14ca2c7 on gustavi:fix-3626 into 96c4562 on zestedesavoir:release-v18.

@Emeric54
Copy link
Contributor

QA => 🆗

@GerardPaligot
Copy link
Member

Merci @Emeric54

@GerardPaligot GerardPaligot merged commit 99f8641 into zestedesavoir:release-v18 May 26, 2016
@gustavi
Copy link
Contributor Author

gustavi commented May 27, 2016

@SpaceFox il faudrait lancer la version que je puisse la déployer sur la beta puis la prod demain :)

@GerardPaligot
Copy link
Member

@gustavi Tu n'as pas les droits nécessaires ?

@SpaceFox
Copy link
Contributor

Il va falloir que tu le fasse, parce que je ne suis pas disponible jusqu'à
dimanche soir…

2016-05-27 5:38 GMT+02:00 Laville Augustin notifications@github.com:

@SpaceFox https://github.com/SpaceFox il faudrait lancer la version que
je puisse la déployer sur la beta puis la prod demain :)

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#3627 (comment),
or mute the thread
https://github.com/notifications/unsubscribe/AFhKnI1vq9u3T1xWeVcSDZFyx5XGRtk2ks5qFmdTgaJpZM4InLaa
.

@gustavi
Copy link
Contributor Author

gustavi commented May 27, 2016

Tu peux juste ma rappeler la procédure exacte ? J'ai dû faire ça il y a plus d'un an la dernière fois. (mp si besoin)

@SpaceFox
Copy link
Contributor

  1. Tu merges la branche release-v18 dans prod en forçant la création d'un
    commit de merge
  2. Tu taggues le commit qui vient d'être créé : « v18-ganesh »
  3. Tu supprimes la branche release-v18
  4. Tu pousses le tout
  5. Tu attends que le build du front arrive. En attendant tu peux vérifier
    si le script de déploiement sur Gandi tape bien sur les bons fichiers.
  6. Tu déploie.

2016-05-27 9:12 GMT+02:00 Laville Augustin notifications@github.com:

Tu peux juste ma rappeler la procédure exacte ? J'ai dû faire ça il y a
plus d'un an la dernière fois. (mp si besoin)

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#3627 (comment),
or mute the thread
https://github.com/notifications/unsubscribe/AFhKnJiLhAdnFFRwXya18ZezYu4hMJ2Hks5qFplUgaJpZM4InLaa
.

gustavi pushed a commit that referenced this pull request Jun 17, 2016
@GerardPaligot @gustavi
Finish v18 (#3666)
5b8e231 
* Enlève le recalcul des votes à la volée lors de la migration

* ZEP-25 : Corrige la commande de migration + instructions MEP

* corrige le MP de la migration

* Rééquilibre le texte de présentation de la home

* Design de la page "Tous les tags"

* Trailing slash des ressources de l'API optionnel

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

* Fix #3494 : correction du menu pour les articles

* Fix #3497 : aligne les messages de la page de connexion

* Fix #3489 : corrige l'affiche des article sur la page d'un membre

* Utilise le bon sprite sur écran rétina

* fix 3513: supprime la supperposition des aides markdown

* fix #3492: Corrige le design des content-item de la home

* Affiche 3 articles et 4 tutos sur la home

Signed-off-by: Quentin Gliech <quentingliech@gmail.com>

* Enlève le nombre des commentaires sur les content-items des tutoriels

Signed-off-by: Quentin Gliech <quentingliech@gmail.com>

* Affiche le nombre de publications sur la home

* corrige une migration d'id

* Corrige la marge sur la home "connectée"

* Corrige la souscription à un sujet

* fix(migration): Corrige les souscriptions des membres.

Closes #3498

* Fix #3507 : correction de l'affichage des noms des filtres

* Typo sur la description sur la page des categories

* fix 3490

* enlève les bots

* Fix #3534 : correction de l'encodage des sorties pour la commande zep-25

* fix: Ordonne correctement les sujets des sujets suivis.

Closes #3536

* fix: Recupere la liste des sujets suivis pour la sidebar.

Closes #3498

* test: Retourne toujours bien les sujets suivis dans le bon ordre.

* fix: N'affiche plus "Suivre ce contenu" pour les visiteurs.

Closes #3545

* Fix #3542 : Empêche de créer un tag en double

* Fix #3533 : corrige le nombre de tuto par tag

* Fix #3496 : correction de la page des contenus (#3524)

* [beta v18]Rend la topbar cohérente en matière de toptags (#3564)

* rend la topbar cohérente

* typo

* typo

* typo

* typo

* typo

* typo

* Mise à jour de la configuration nginx

* [v18]Donne les instruction pour corriger #3448 (#3577)

Donne les instruction pour supprimer les anciennes tables du module des tutos #3448

* [ci skip] Correction du message de migration de la ZEP-25 (#3595)

* Fix #3592: Ajout du lien vers "CONTRIBUTING.md" (#3596)

* Fix #3587 : Index les relations generiques des modeles des notifs. (#3594)

* fix(notif): Index les relations generiques des modeles des notifs.

Pour information, Django n'index pas automatiquement les relations
generiques ce qui vient plomber les performances quand on
commence a disposer d'une grosse base de donnees.

Voir https://groups.google.com/forum/#!topic/django-users/GJ-FVh40-F4

Closes #3587

* [ci skip] Ajoute la commande d'opti mysql dans le update.md

* Fix #3584 : correction des commandes de migration de la v18 (#3586)

* Évite les tags répétés plusieurs fois (#3601)

* augmente la taille des tags et évite qu'ils soient recréés quand ils sont trop grands.

* ajoute un test unitaire

* Fix l'unicite des tags

* Corrige l'unicite des tags (again)

* change le processus de création des tags lors du sujet de beta

* fix indexerror

* pep8

* pep8

* fix indexerror

* remove useless test

* fix test

* pep8

* Correction affichage du tag (#3597)

* ZEP-25 : correction mineure de la commande de migration (#3612)

* Fix #3614 - ZEP-25 : corrige les URL dans le mp de migration (#3616)

* Fix #3626 : corrige les espaces dans l'ajout de tags sur les contenus (#3627)

* Hotfix v18 : votes negatifs (#3635)

* fix(notif): Marque comme lu une notif d'un contenu non suivi. (#3664)

Closes #3659

* fix(notif): Evite les notifs persistantes a la suppression. (#3663)

Closes #3639

* fix(3630): Migration des souscriptions par e-mail. (#3661)

Closes #3630

* fix(notif): Mise a jour de la date des notifications. (#3662)

Closes #3628
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@gustavi @coveralls @Emeric54 @GerardPaligot @SpaceFox