Fix #3496 : corrige la page des contenus #3524
Conversation
|
| urlpatterns = [ | ||
| # Flux | ||
| url(r'^flux/rss/$', LastContentFeedRSS(), name='feed-rss'), |
There was a problem hiding this comment.
Pas correct : il faut utiliser as_view.
There was a problem hiding this comment.
J'ai gardé la même syntaxe que dans urls_tutorials.py et dans urls_articles.py. S'il faut faire un refactor aussi pour les flux, je ne pense pas être capable de le faire proprement (je débute avec Django).
There was a problem hiding this comment.
Pour le coup @artragis c'est un tout petit peu trompé, pas besoin de as_view ;)
|
Pour les tags, je ne sais pas non plus... J'ai supposé que le comportement attendu était bien d'arriver sur la page "Tous les contenus" en cliquant sur "Tous les tags" dans les tutos. A voir avec quelqu'un qui a suivi la ZEP-25 (si je ne me trompe pas). Ping @gustavi ? |
|
Je pense pas que ça soit voulu : on est dans le sous-menu "tutoriels" Le 09/04/2016 16:07, Vulser a écrit :
|
|
A mon sens, il n'est pas souhaitable d'arriver sur une page avec le concept de contenu qui est inconnu pour un nouveau sur Zeste de Savoir (voir même pour des habitués). Aujourd'hui, techniquement, nous avon des articles, des tutoriels et des contenus.Mais fonctionnellement, nous avons des tutoriels et des articles. |
|
Techniquement, nous n'avons que des contenus, fonctionnellement des D'ailleurs on remarqueras que la liste des "tags populaire" est aussi un En tout cas, je pense que changer le template du menu pour mettre l'url Le 09/04/2016 17:17, Gérard Paligot a écrit :
|
|
Désolé d'avoir mis du temps à répondre, j'avais loupé ce topic.
Je crois que c'était spécifié dans la ZEP que ça devait être comme cela. Après on peut aviser mais je crois que l'idée était de rapprocher les contenus, en particulier avec les tags. On peut changer toutesfois si ça ne convient pas. ping @SpaceFox : t'en penses quoi ? |
|
Il faudrait vérifier la ZEP, mais cette histoire d'utiliser les tags pour rapprocher les contenus me dit quelque chose. |
|
LA ZEP dit :
|
|
Ok donc je modifie une de mes PR où j'avais restreint les choses. Le 19 avril 2016 à 09:37, Laville Augustin notifications@github.com a
|
|
@gustavi Attention. Ok, la liste des contenus semblent souhaitée mais il faut réfléchir à une navigation qui ne va pas perdre l'utilisateur. A l'époque où j'avais fais mes tests sur la bêta, j'arrivais parfois sur des pages dédiées aux tutoriels/articles, parfois à des pages dédiées à tous les contenus mais en cliquant sur des choses similaires dans des contextes non ambigus (c'est-à-dire, en cliquant sur une catégorie/tag d'un tutoriel ou d'un article). Les pages listant tous les contenus d'une catégorie ou d'un tag doit provenir d'un contexte commun aussi. |
|
@Vulser Nous en sommes où dans cette issue ? |
|
J'avoue être perdu par rapport au résultat attendu. C'est vrai qu'en l'état, la navigation n'est pas optimale. Mais je n'aurai pas le temps de me replonger dedans ces semaines-ci... Donc si quelqu'un veut reprendre la PR, pas de soucis, j'en serais ravi ;) |
|
A priori, si c'est quelque chose de voulu, j'avais un QA ok. Sinon, il suffira de changer le lien dans le bandeau. Donc je suis pour merger et attendre la béta pour voir si on décide de changer le comportement. |
|
Pour moi c'est good aussi. |
* 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
QA
NB : La PR #3516 corrige déjà le bug avec les tags/catégories dans le fil d'Ariane.