[v18]corrige le nombre de tuto par tag #3533

artragis · 2016-04-10T20:44:42Z

Q	R
Type de modification	correction de bug / nouvelle fonctionnalité / évolution
Ticket(s) (issue(s)) concerné(s)	(#3495)

je reprends #3518

QA

Dans la page de tous les tags, sélectionnez un tag, retenez le nombre de tutos qu'il y a à l'interrieur
Naviguer sur un des tutos de ce tag
Modifiez son titre
publiez la nouvelle version
observez dans la page de tous les tags que rien n'a changé

SQASH A VENIR.

coveralls · 2016-04-10T20:55:25Z

Coverage increased (+34.1%) to 87.24% when pulling 31a5713 on artragis:patch-8 into 464434f on zestedesavoir:release-v18.

coveralls · 2016-04-10T20:59:06Z

Coverage decreased (-0.004%) to 53.139% when pulling 31a5713 on artragis:patch-8 into 464434f on zestedesavoir:release-v18.

coveralls · 2016-04-11T06:10:16Z

Coverage increased (+34.1%) to 87.241% when pulling 490bf2b on artragis:patch-8 into 464434f on zestedesavoir:release-v18.

coveralls · 2016-04-11T10:44:30Z

Coverage increased (+34.1%) to 87.241% when pulling 158d159 on artragis:patch-8 into 464434f on zestedesavoir:release-v18.

gustavi · 2016-04-14T06:02:53Z

NEED QA

Emeric54 · 2016-04-17T18:45:26Z

QA : ❌

Il m'est impossible de me rendre sur la page des tags :

Internal Server Error: /contenus/tags/
Traceback (most recent call last):
  File "/home/emeric/zdsenv/local/lib/python2.7/site-packages/django/core/handlers/base.py", line 132, in get_response
    response = wrapped_callback(request, *callback_args, **callback_kwargs)
  File "/home/emeric/zdsenv/local/lib/python2.7/site-packages/django/views/generic/base.py", line 71, in view
    return self.dispatch(request, *args, **kwargs)
  File "/home/emeric/zdsenv/local/lib/python2.7/site-packages/django/views/generic/base.py", line 89, in dispatch
    return handler(request, *args, **kwargs)
  File "/home/emeric/zdsenv/local/lib/python2.7/site-packages/django/views/generic/list.py", line 159, in get
    self.object_list = self.get_queryset()
  File "/home/emeric/zds-site/zds/tutorialv2/views/views_published.py", line 681, in get_queryset
    publishablecontent__type__in=self.displayed_types)\
  File "/home/emeric/zdsenv/local/lib/python2.7/site-packages/django/db/models/manager.py", line 127, in manager_method
    return getattr(self.get_queryset(), name)(*args, **kwargs)
  File "/home/emeric/zdsenv/local/lib/python2.7/site-packages/django/db/models/query.py", line 679, in filter
    return self._filter_or_exclude(False, *args, **kwargs)
  File "/home/emeric/zdsenv/local/lib/python2.7/site-packages/django/db/models/query.py", line 697, in _filter_or_exclude
    clone.query.add_q(Q(*args, **kwargs))
  File "/home/emeric/zdsenv/local/lib/python2.7/site-packages/django/db/models/sql/query.py", line 1310, in add_q
    clause, require_inner = self._add_q(where_part, self.used_aliases)
  File "/home/emeric/zdsenv/local/lib/python2.7/site-packages/django/db/models/sql/query.py", line 1338, in _add_q
    allow_joins=allow_joins, split_subq=split_subq,
  File "/home/emeric/zdsenv/local/lib/python2.7/site-packages/django/db/models/sql/query.py", line 1200, in build_filter
    lookups, value)
  File "/home/emeric/zdsenv/local/lib/python2.7/site-packages/django/db/models/fields/related.py", line 1774, in get_lookup_constraint
    raise TypeError('Related Field got invalid lookup: %s' % lookup_type)
TypeError: Related Field got invalid lookup: is_null

coveralls · 2016-04-17T19:16:09Z

Coverage increased (+34.1%) to 87.287% when pulling 9b4f21c on artragis:patch-8 into 464434f on zestedesavoir:release-v18.

coveralls · 2016-04-18T06:07:25Z

Coverage increased (+34.1%) to 87.272% when pulling 6af2872 on artragis:patch-8 into 464434f on zestedesavoir:release-v18.

artragis · 2016-04-18T09:19:19Z

Corrigé @Emeric54

…un tag

coveralls · 2016-04-19T07:59:02Z

Coverage increased (+16.4%) to 87.272% when pulling d826a33 on artragis:patch-8 into fb73363 on zestedesavoir:release-v18.

coveralls · 2016-04-19T08:14:37Z

Coverage increased (+16.4%) to 87.272% when pulling d826a33 on artragis:patch-8 into fb73363 on zestedesavoir:release-v18.

GerardPaligot · 2016-04-25T16:09:30Z

@Emeric54 Tu aurais le temps de re-QA rapidement cette PR ?

Emeric54 · 2016-04-25T16:25:22Z

Pas avant ce week-end, malheureusement.
Le 25 avr. 2016 18:09, "Gérard Paligot" notifications@github.com a écrit :

@Emeric54 https://github.com/Emeric54 Tu aurais le temps de re-QA
rapidement cette PR ?

—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
#3533 (comment)

gustavi · 2016-04-26T07:23:05Z

Pas bon, j'ai toujours plus que le nombre voulu.

artragis · 2016-04-26T07:27:49Z

Peux-tu donner :

une capture d'écran
le nombre voulu

Car là je peux pas te dire quoi que ce soit de plus.

2016-04-26 9:23 GMT+02:00 Laville Augustin notifications@github.com:

Pas bon, j'ai toujours plus que le nombre voulu.

—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
#3533 (comment)

gustavi · 2016-04-26T07:30:52Z

Tous les tags : http://i.imgur.com/0ZX51wJ.png et
Article publié puis modifé et re-publié : http://i.imgur.com/f9D0pbf.png
Autre article : http://i.imgur.com/anIdOk2.png

Attendu : 2 2 1 1 1 1 et non 3 3 1 1 1 1

artragis · 2016-04-26T07:37:34Z

As-tu un tutoriel publié sous le tag c++ ou python?

2016-04-26 9:30 GMT+02:00 Laville Augustin notifications@github.com:

Tous les tags : http://i.imgur.com/0ZX51wJ.png et

Article publié puis modifé et re-publié :
http://i.imgur.com/f9D0pbf.png

Autre article : http://i.imgur.com/anIdOk2.png

Attendu : 2 2 1 1 1 1 et non 3 3 1 1 1 1

—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
#3533 (comment)

gustavi · 2016-04-26T07:44:12Z

Les deux seuls contenus publiés avec des tags sont les deux précédents.

artragis · 2016-04-26T07:53:59Z

j'ai fait un "essai de la dernière chance".

2016-04-26 9:44 GMT+02:00 Laville Augustin notifications@github.com:

Les deux seuls contenus publiés avec des tags sont les deux précédents.

—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
#3533 (comment)

gustavi · 2016-04-26T07:59:19Z

Pas encore terminé la QA mais j'ai http://i.imgur.com/jdCEmK2.png et http://i.imgur.com/DkTkWzM.png en cliquant sur la page d'un tag

artragis · 2016-04-26T08:01:19Z

C'est pas la même issue et je ne viens que de commiter mon fix de la
dernière chance (typo dans mon mdp :p)

2016-04-26 9:59 GMT+02:00 Laville Augustin notifications@github.com:

Pas encore terminé la QA mais j'ai http://i.imgur.com/jdCEmK2.png et
http://i.imgur.com/DkTkWzM.png en cliquant sur la page d'un tag

—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
#3533 (comment)

gustavi · 2016-04-26T08:04:43Z

\o/

QA OK : je merge (quand Travis a terminé) !

coveralls · 2016-04-26T08:13:54Z

Coverage increased (+16.4%) to 87.288% when pulling 038ecbb on artragis:patch-8 into fb73363 on zestedesavoir:release-v18.

* 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

artragis changed the title ~~reprise de la PR de gustavi~~ [v18]corrige le nombre de tuto par tag Apr 11, 2016

artragis mentioned this pull request Apr 11, 2016

Fix #3495 : corrige le nombre de contenus spécifié pour un tag #3518

Closed

artragis force-pushed the patch-8 branch 4 times, most recently from 50e10a4 to 158d159 Compare April 11, 2016 10:33

artragis force-pushed the patch-8 branch from 6af2872 to 354af29 Compare April 19, 2016 07:39

Fix zestedesavoir#3495 : corrige le nombre de contenus spécifié pour …

d826a33

…un tag

artragis force-pushed the patch-8 branch from 354af29 to d826a33 Compare April 19, 2016 07:44

GerardPaligot mentioned this pull request Apr 25, 2016

Incohérence dans les tags les plus utilisés #3493

Closed

gustavi added the Bloquant Ticket qui doit être traité avant la prochaine mise à jour label Apr 26, 2016

refix

038ecbb

gustavi merged commit 4437f26 into zestedesavoir:release-v18 Apr 26, 2016

artragis deleted the patch-8 branch April 13, 2017 07:26

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[v18]corrige le nombre de tuto par tag #3533

[v18]corrige le nombre de tuto par tag #3533

artragis commented Apr 10, 2016

coveralls commented Apr 10, 2016

coveralls commented Apr 10, 2016

coveralls commented Apr 11, 2016

coveralls commented Apr 11, 2016

gustavi commented Apr 14, 2016

Emeric54 commented Apr 17, 2016

coveralls commented Apr 17, 2016

coveralls commented Apr 18, 2016

artragis commented Apr 18, 2016

coveralls commented Apr 19, 2016

coveralls commented Apr 19, 2016

GerardPaligot commented Apr 25, 2016

Emeric54 commented Apr 25, 2016

gustavi commented Apr 26, 2016

artragis commented Apr 26, 2016

gustavi commented Apr 26, 2016

artragis commented Apr 26, 2016

gustavi commented Apr 26, 2016

artragis commented Apr 26, 2016

gustavi commented Apr 26, 2016

artragis commented Apr 26, 2016

gustavi commented Apr 26, 2016 •

edited

Loading

coveralls commented Apr 26, 2016 •

edited

Loading

[v18]corrige le nombre de tuto par tag #3533

[v18]corrige le nombre de tuto par tag #3533

Conversation

artragis commented Apr 10, 2016

QA

coveralls commented Apr 10, 2016

coveralls commented Apr 10, 2016

coveralls commented Apr 11, 2016

coveralls commented Apr 11, 2016

gustavi commented Apr 14, 2016

Emeric54 commented Apr 17, 2016

coveralls commented Apr 17, 2016

coveralls commented Apr 18, 2016

artragis commented Apr 18, 2016

coveralls commented Apr 19, 2016

coveralls commented Apr 19, 2016

GerardPaligot commented Apr 25, 2016

Emeric54 commented Apr 25, 2016

gustavi commented Apr 26, 2016

artragis commented Apr 26, 2016

gustavi commented Apr 26, 2016

artragis commented Apr 26, 2016

gustavi commented Apr 26, 2016

artragis commented Apr 26, 2016

gustavi commented Apr 26, 2016

artragis commented Apr 26, 2016

gustavi commented Apr 26, 2016 • edited Loading

coveralls commented Apr 26, 2016 • edited Loading

gustavi commented Apr 26, 2016 •

edited

Loading

coveralls commented Apr 26, 2016 •

edited

Loading