Ajoute une alerte sur le nombre de contenus orphelins en validation #4212

gllmc · 2017-02-14T21:19:29Z

Q	R
Type de modification	nouvelle fonctionnalité
Ticket(s) (issue(s)) concerné(s)	#4210

Cette PR ajoute au menu d'un membre staff le nombre de contenus orphelins en attente de validation.

QA

Reconstruire le front
Mettre un tutoriel en validation ;
Vérifier la présence du (1) dans le menu ;
Réserver le tutoriel ;
Vérifier que le (1) a disparu ;
Idem avec les articles.

coveralls · 2017-02-14T21:30:40Z

Coverage increased (+0.004%) to 88.371% when pulling c03963b on GCodeur:alerte_contenus_attente into 4612e11 on zestedesavoir:dev.

pierre-24 · 2017-02-15T19:34:29Z

zds/utils/templatetags/interventions.py

+
+
+@register.filter(name='waiting_tutorials_count')
+def waiting_tutorials_count(user):


Pourquoi ce paramètre user ?

Pour la même raison que pour la fonction qui récupère les alertes : afin de pouvoir appliquer le filtre sur un objet User. ;)

... Ce qui ne sert à rien dans ta fonction (ce qui est le cas pour les alertes, aussi, mais une issue à la fois) ;) Et à réflexion, pourquoi pas plutôt un paramètre type, qui te permetrait de faire "tutorial"|waiting_count, un truc du genre :)

J'aime bien cette solution, va pour ça.

pierre-24 · 2017-02-15T19:35:55Z

Sinon, QA ok :

pierre-24 · 2017-02-15T19:43:16Z

Oh, et n'oublie pas d'éditer la documentation des templatetags, merci :)

coveralls · 2017-02-15T20:07:55Z

Coverage increased (+0.002%) to 88.369% when pulling 0e92d1d on GCodeur:alerte_contenus_attente into 99e47cb on zestedesavoir:dev.

gllmc · 2017-02-15T20:11:31Z

Je pense que tout est bon, @pierre-24 :)

pierre-24 · 2017-02-15T20:35:34Z

zds/utils/templatetags/interventions.py

+
+@register.filter(name='waiting_count')
+def waiting_count(content_type):
+    return Validation.objects.filter(


Je vais faire mon chiant, et demander une petite levée d'erreur si content_type not in zds.tutorialsv2.models.TYPE_CHOICES_DICT. Pour être sur :)

pierre-24 · 2017-02-15T21:19:45Z

zds/utils/templatetags/interventions.py

+
+@register.filter(name='waiting_count')
+def waiting_count(content_type):
+    if not content_type in TYPE_CHOICES_DICT:


coveralls · 2017-02-16T09:53:19Z

Coverage decreased (-0.002%) to 88.365% when pulling 2ce0388 on GCodeur:alerte_contenus_attente into 99e47cb on zestedesavoir:dev.

firm1 · 2017-02-17T11:06:28Z

Au vu du rendu, j'avoue que ce n'est pas a ça que je me m'attendais visuellement.

Pour l'instant le compteur est encore caché je trouve, car il faut cliquer sur le menu (action que je ne fais quasi jamais en tant que staff, sauf quand j'ai envie justement d'aller en zone de validation). Je voyais plutôt quelque chose à droite des alertes de modération directement visible.

gllmc · 2017-02-17T11:20:26Z

C'est effectivement à ça que j'avais pensé au début, mais on en avait un peu parlé sur IRC et on avait choisi ça (notamment pour des raisons de place il me semble). En soi, ça ne me paraît pas impossible à envisager, mais :

à voir en terme de place, notamment quand le lien vers les tribunes sera ajouté
est-ce que ce ne sera pas un peu anti-ergonomique d'avoir trois icônes ouvrant un menu déroulant et une redirigeant vers une autre page ?
je ne maîtrise pas du tout la façon dont les icônes sont gérées et en bidouillant le HTML, je n'ai pas trouvé d'icône correspondante :/

firm1 · 2017-02-17T13:10:25Z

Le problème avec la solution actuelle c'est que l'effet escompté (une alerte que l'on voit en permanence) n'est pas atteint.

à voir en terme de place, notamment quand le lien vers les tribunes sera ajouté

Si on a un problème de place, on peut au moins partir du principe que le nouveau bouton d'alerte ne sera visible que sur les grands écrans. On peut conserver le travail que tu as fais dans cette PR pour les petit écrans.

est-ce que ce ne sera pas un peu anti-ergonomique d'avoir trois icônes ouvrant un menu déroulant et une redirigeant vers une autre page ?

On peut avoir une icône qui affiche le nombre de contenus orphelins d'une part. En cliquant dessus on a une liste dans laquelle on a les orphelins de type article d'une part et les orphelins de type tutoriels. Et plus tard on aura aussi les tribunes. ça ne me semble pas déconnant.

je ne maîtrise pas du tout la façon dont les icônes sont gérées et en bidouillant le HTML, je n'ai pas trouvé d'icône correspondante

A mon sens il faudrait quelque chose qui évoque la validation ou le contenu. Mais en gros pour rajouter une nouvelle icone il faut le rajouter dans le dossier des assets et s'inspirer de la partie front dans le code (malheureusement on a pas encore de doc là dessus)

gllmc · 2017-02-23T20:36:27Z

Histoire d'avancer un peu, je viens de tenter un autre moyen de faire qui est d'afficher l'icône des paramètres aussi pour les membres du staff (je parle de la roue dentée) et ainsi de pouvoir y mettre le nombre comme on le fait actuellement pour les alertes par exemple. Seulement, niveau rendu, ça donne ça :

Lorsque je supprime par exemple l'icône des notifications via les outils de développement, tout revient à la normale. J'ai donc l'impression que le CSS n'accepte que 3 icônes (plus l'avatar), mais je ne le maîtrise pas du tout et ne sais pas comment y remédier.

Sinon, voici le rendu (note qu'il faut trouver un moyen de remettre l'icônes des notifications), est-ce que ça t'irait ?

coveralls · 2017-04-03T20:04:35Z

Coverage decreased (-0.002%) to 88.64% when pulling 6281072 on GCodeur:alerte_contenus_attente into d526faa on zestedesavoir:dev.

"There is nothing that a bit of CSS cannot fix" !

gllmc · 2017-04-04T08:21:40Z

Problème résolu grâce à @pierre-24, l'alerte est maintenant fixée sur l'avatar, voici ce que ça donne :

coveralls · 2017-04-04T08:30:44Z

Coverage increased (+0.0002%) to 88.642% when pulling 472ba3d on GCodeur:alerte_contenus_attente into d526faa on zestedesavoir:dev.

coveralls · 2017-04-04T08:45:19Z

Coverage increased (+0.0002%) to 88.642% when pulling abce88b on GCodeur:alerte_contenus_attente into d526faa on zestedesavoir:dev.

coveralls · 2017-04-04T09:15:49Z

Coverage increased (+0.0002%) to 88.642% when pulling fcc8d99 on GCodeur:alerte_contenus_attente into d526faa on zestedesavoir:dev.

artragis · 2017-04-10T17:57:45Z

@pierre-24 tu valides la QA?

pierre-24 · 2017-04-10T18:42:17Z

Ouaip' :p

pierre-24 added C-Front Concerne l'interface du site QA svp labels Feb 15, 2017

pierre-24 suggested changes Feb 15, 2017

View reviewed changes

gllmc added 6 commits April 3, 2017 21:43

Ajoute une alerte sur le nombre de contenus orphelins en validation

24a8da0

QA de pierre-24

4155def

Vérifie content_type

39a5da1

tutorialsv2 => tutorialv2

ea50e38

Remarque de pierre-24 (PEP-8)

3e6d8d2

Erreur lors du rebase

6281072

pierre-24 and others added 2 commits April 4, 2017 09:29

Nothing that a bit of CSS cannot fix

d5b6509

Merge pull request #10 from pierre-24/add_some_CSS

472ba3d

"There is nothing that a bit of CSS cannot fix" !

pierre-24 approved these changes Apr 4, 2017

View reviewed changes

Adapte la doc en conséquence

abce88b

Doc : erreur sur le marquage du code

fcc8d99

gllmc mentioned this pull request Apr 10, 2017

Dernier appel pour la v23 #4278

Closed

3 tasks

artragis merged commit b02adc6 into zestedesavoir:dev Apr 10, 2017

gllmc added this to the Version de développement milestone Apr 10, 2017

gllmc removed the QA svp label Apr 10, 2017

gllmc deleted the alerte_contenus_attente branch July 17, 2017 11:53

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Ajoute une alerte sur le nombre de contenus orphelins en validation #4212

Ajoute une alerte sur le nombre de contenus orphelins en validation #4212

gllmc commented Feb 14, 2017 •

edited

coveralls commented Feb 14, 2017 •

edited

pierre-24 Feb 15, 2017

gllmc Feb 15, 2017

pierre-24 Feb 15, 2017

gllmc Feb 15, 2017

pierre-24 commented Feb 15, 2017

pierre-24 commented Feb 15, 2017

coveralls commented Feb 15, 2017 •

edited

gllmc commented Feb 15, 2017

pierre-24 Feb 15, 2017 •

edited

pierre-24 Feb 15, 2017

coveralls commented Feb 16, 2017 •

edited

firm1 commented Feb 17, 2017

gllmc commented Feb 17, 2017

firm1 commented Feb 17, 2017

gllmc commented Feb 23, 2017

coveralls commented Apr 3, 2017 •

edited

gllmc commented Apr 4, 2017

coveralls commented Apr 4, 2017 •

edited

coveralls commented Apr 4, 2017 •

edited

coveralls commented Apr 4, 2017 •

edited

artragis commented Apr 10, 2017

pierre-24 commented Apr 10, 2017



		@register.filter(name='waiting_tutorials_count')
		def waiting_tutorials_count(user):

Ajoute une alerte sur le nombre de contenus orphelins en validation #4212

Ajoute une alerte sur le nombre de contenus orphelins en validation #4212

Conversation

gllmc commented Feb 14, 2017 • edited

QA

coveralls commented Feb 14, 2017 • edited

pierre-24 Feb 15, 2017

Choose a reason for hiding this comment

gllmc Feb 15, 2017

Choose a reason for hiding this comment

pierre-24 Feb 15, 2017

Choose a reason for hiding this comment

gllmc Feb 15, 2017

Choose a reason for hiding this comment

pierre-24 commented Feb 15, 2017

pierre-24 commented Feb 15, 2017

coveralls commented Feb 15, 2017 • edited

gllmc commented Feb 15, 2017

pierre-24 Feb 15, 2017 • edited

Choose a reason for hiding this comment

pierre-24 Feb 15, 2017

Choose a reason for hiding this comment

coveralls commented Feb 16, 2017 • edited

firm1 commented Feb 17, 2017

gllmc commented Feb 17, 2017

firm1 commented Feb 17, 2017

gllmc commented Feb 23, 2017

coveralls commented Apr 3, 2017 • edited

gllmc commented Apr 4, 2017

coveralls commented Apr 4, 2017 • edited

coveralls commented Apr 4, 2017 • edited

coveralls commented Apr 4, 2017 • edited

artragis commented Apr 10, 2017

pierre-24 commented Apr 10, 2017

gllmc commented Feb 14, 2017 •

edited

coveralls commented Feb 14, 2017 •

edited

coveralls commented Feb 15, 2017 •

edited

pierre-24 Feb 15, 2017 •

edited

coveralls commented Feb 16, 2017 •

edited

coveralls commented Apr 3, 2017 •

edited

coveralls commented Apr 4, 2017 •

edited

coveralls commented Apr 4, 2017 •

edited

coveralls commented Apr 4, 2017 •

edited