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

Ajoute une page avec les derniers sujets publiés #4831

Merged
merged 23 commits into from Jan 30, 2018

Conversation

Projects
None yet
7 participants
@rezemika
Contributor

rezemika commented Jan 3, 2018

Cette PR fait suite à une suggestion faite sur le forum (ici). Il faudrait en discuter et voir si elle pertinente ou non.

Ce commit ajoute une page accessible via l'URL "/forums/derniers-sujets/" qui affiche les derniers sujets publiés, toutes catégories confondues. Le bouton "Tous les sujets" sur la page d'accueil est renommé en "Derniers sujets" et pointe maintenant vers cette page.

La pagination n'y a pas été ajoutée étant donné que cette page a davantage pour but de voir quels sont les derniers sujets publiés que d'explorer tous les sujets publiés un jour sur un des forums (seuls les sujets les plus récents sont pertinents).
Le nombre de sujets affichés sur la page correspond au nombre de sujets qui seraient affichés sur la première page d'une liste avec pagination (voir settings.ZDS_APP['forum']['topics_per_page']).

La catégorie (et un lien vers celle-ci) est également ajouté dans les blocs des sujets, afin de rappeler que cette page liste les derniers sujets quelque soit leur catégorie (ils sont ordonnés selon leur date de publication).

Numéro du ticket concerné : aucun

Points à vérifier / débattre

  • URL de la page ("/forums/derniers-sujets/" pour l'instant) ;
  • Mise en forme de la page ;
  • Nom du bouton vers cette page sur la page d'accueil ("Derniers sujets" pour l'instant, un peu redondant avec le titre à côté) Désormais "En voir plus" ;
  • Pagination ?
  • Icônes pour les boutons de tri.

@Situphen Situphen added the Feedback label Jan 5, 2018

@motet-a motet-a changed the title from [Feedback] [WIP] Ajoute une page avec les derniers sujets publiés to [WIP] Ajoute une page avec les derniers sujets publiés Jan 6, 2018

@gcodeur

Merci pour cette PR, je trouve cette nouveauté pertinente !

  • Il y a une faille de sécurité à corriger (les sujets privés sont affichés).
  • Éventuellement dans une deuxième itération, je pense qu'un filtre sur la gauche permettant de voir les derniers sujets sur la base de la date du dernier message serait pertinent.
  • Je mettrais bien un lien Tous les forums en haut du menu de gauche sur cette page.
Show outdated Hide outdated templates/forum/last_subjects.html Outdated
Show outdated Hide outdated templates/home.html Outdated
Show outdated Hide outdated zds/forum/views.py Outdated
Show outdated Hide outdated templates/forum/last_subjects.html Outdated
Show outdated Hide outdated templates/forum/last_subjects.html Outdated
@rezemika

This comment has been minimized.

Show comment
Hide comment
@rezemika

rezemika Jan 11, 2018

Contributor

Merci pour ton retour @gcodeur !

J'ai corrigé les erreurs, transformé la vue en fonction (je ne voyais pas comment gérer le filtrage des sujets visible autrement) et j'ai "préparé le terrain" pour ajouter les boutons de changement d'affichage.

Par contre, je ne te cache pas que je risque d'avoir un peu de mal à gérer cette partie là (mais je ne demande qu'à essayer :) ).

Contributor

rezemika commented Jan 11, 2018

Merci pour ton retour @gcodeur !

J'ai corrigé les erreurs, transformé la vue en fonction (je ne voyais pas comment gérer le filtrage des sujets visible autrement) et j'ai "préparé le terrain" pour ajouter les boutons de changement d'affichage.

Par contre, je ne te cache pas que je risque d'avoir un peu de mal à gérer cette partie là (mais je ne demande qu'à essayer :) ).

@gcodeur

Par contre, je ne te cache pas que je risque d'avoir un peu de mal à gérer cette partie là (mais je ne demande qu'à essayer :) ).

Le filtre fonctionnera comme ceux qui sont utilisés dans les pages de forums. L'idée est donc de passer par un paramètre GET order pour avoir une URL comme /forums/derniers-sujets/?order=last_post.

Selon ce paramètre, tu appliques ton order_by (dans la vue) sur -pubdate ou -last_message__pubdate) et tu modifies le filtre qui apparaît comme sélectionné dans le template.

Conseil : tu peux prévoir deux valeurs possibles pour order : creation et last_post. Si le paramètre est inexistant ou que sa valeur est incorrecte, considère que creation est sélectionné (c'est la valeur par défaut).

Show outdated Hide outdated zds/forum/views.py Outdated
Show outdated Hide outdated templates/forum/last_subjects.html Outdated
@rezemika

This comment has been minimized.

Show comment
Hide comment
@rezemika

rezemika Jan 11, 2018

Contributor

Merci pour ton aide !

Je viens de tester en local et ça m'a l'air d'être bon. :)

Au passage, il faudrait peut être trouver une autre icône pour les boutons filtres ? Je penses aux classes move (deux flèches verticales tête-bêche, utilisées pour la modale de réordonnancent dans un contenu) ou history (une horloge). Qu'en pense-tu ?

Contributor

rezemika commented Jan 11, 2018

Merci pour ton aide !

Je viens de tester en local et ça m'a l'air d'être bon. :)

Au passage, il faudrait peut être trouver une autre icône pour les boutons filtres ? Je penses aux classes move (deux flèches verticales tête-bêche, utilisées pour la modale de réordonnancent dans un contenu) ou history (une horloge). Qu'en pense-tu ?

@rezemika

This comment has been minimized.

Show comment
Hide comment
@rezemika

rezemika Jan 13, 2018

Contributor

À votre avis, je mets quelle classe pour les icônes des boutons de tri ? move (deux flèches tête-bêche) ou history (une horloge), ou autre chose ?

Contributor

rezemika commented Jan 13, 2018

À votre avis, je mets quelle classe pour les icônes des boutons de tri ? move (deux flèches tête-bêche) ou history (une horloge), ou autre chose ?

Show outdated Hide outdated zds/forum/views.py Outdated
Show outdated Hide outdated zds/forum/views.py Outdated
@rezemika

This comment has been minimized.

Show comment
Hide comment
@rezemika

rezemika Jan 14, 2018

Contributor

Merci pour ton retour ! Voilà les modifs ! :)

Pour l'obtention des topics lisibles par l'utilisateur, j'ai créé une nouvelle méthode dans Profile pour obtenir la liste des forums accessibles. Ça permet de l'utiliser directement dans des requêtes.

Contributor

rezemika commented Jan 14, 2018

Merci pour ton retour ! Voilà les modifs ! :)

Pour l'obtention des topics lisibles par l'utilisateur, j'ai créé une nouvelle méthode dans Profile pour obtenir la liste des forums accessibles. Ça permet de l'utiliser directement dans des requêtes.

@coveralls

This comment has been minimized.

Show comment
Hide comment
@coveralls

coveralls Jan 14, 2018

Coverage Status

Coverage decreased (-0.03%) to 89.602% when pulling 3725edd on rezemika:last-subjects-page into 2afc278 on zestedesavoir:dev.

coveralls commented Jan 14, 2018

Coverage Status

Coverage decreased (-0.03%) to 89.602% when pulling 3725edd on rezemika:last-subjects-page into 2afc278 on zestedesavoir:dev.

@zestedesavoir zestedesavoir deleted a comment from coveralls Jan 19, 2018

@zestedesavoir zestedesavoir deleted a comment from coveralls Jan 19, 2018

@zestedesavoir zestedesavoir deleted a comment from coveralls Jan 19, 2018

@zestedesavoir zestedesavoir deleted a comment from coveralls Jan 19, 2018

@zestedesavoir zestedesavoir deleted a comment from coveralls Jan 19, 2018

@zestedesavoir zestedesavoir deleted a comment from coveralls Jan 19, 2018

@zestedesavoir zestedesavoir deleted a comment from coveralls Jan 19, 2018

@zestedesavoir zestedesavoir deleted a comment from coveralls Jan 19, 2018

@coveralls

This comment has been minimized.

Show comment
Hide comment
@coveralls

coveralls Jan 19, 2018

Coverage Status

Coverage increased (+0.009%) to 89.687% when pulling 3c00ec5 on rezemika:last-subjects-page into 02db935 on zestedesavoir:dev.

coveralls commented Jan 19, 2018

Coverage Status

Coverage increased (+0.009%) to 89.687% when pulling 3c00ec5 on rezemika:last-subjects-page into 02db935 on zestedesavoir:dev.

@gcodeur

Je pense que ce serait bien d'avoir un test unitaire sur cette page qui vérifierait :

  • que la page fonctionne sans erreur chez les utilisateurs connectés et non connectés.
  • que les forums inaccessibles ne sont pas affichés.

Tu saurais comment faire ça ?

Show outdated Hide outdated zds/forum/views.py Outdated
@rezemika

This comment has been minimized.

Show comment
Hide comment
@rezemika

rezemika Jan 21, 2018

Contributor

C'est bon, les tests unitaires sont écrits et m'ont l'air de passer en local. Qu'en pense-tu ?

Contributor

rezemika commented Jan 21, 2018

C'est bon, les tests unitaires sont écrits et m'ont l'air de passer en local. Qu'en pense-tu ?

Show outdated Hide outdated templates/forum/last_subjects.html Outdated
@rezemika

This comment has been minimized.

Show comment
Hide comment
@rezemika

rezemika Jan 28, 2018

Contributor

Plus qu'à trouver des icônes adaptées pour les boutons de tri et je pense que ce sera prêt à être mergé. Je suis preneur de toute suggestion. :)

@gcodeur : Les tests sont faits et passent sans problème. ;)

Contributor

rezemika commented Jan 28, 2018

Plus qu'à trouver des icônes adaptées pour les boutons de tri et je pense que ce sera prêt à être mergé. Je suis preneur de toute suggestion. :)

@gcodeur : Les tests sont faits et passent sans problème. ;)

rezemika added some commits Jan 3, 2018

Ajoute une page avec les derniers sujets publiés
Ce commit ajoute une page accessible via l'URL "/forums/derniers-sujets/" qui affiche les derniers sujets publiés, toutes catégories confondues. Le bouton "Tous les sujets" sur la page d'accueil est renommé en "Derniers sujets" et pointe maintenant vers cette page.

La pagination n'y a pas été ajoutée étant donné que cette page a davantage pour but de voir quels sont les derniers sujets publiés que d'explorer tous les sujets publiés un jour sur un des forums (seuls les sujets les plus récents sont pertinents).
Le nombre de sujets affichés sur la page correspond au nombre de sujets qui seraient affichés sur la première page d'une liste avec pagination (voir `settings.ZDS_APP['forum']['topics_per_page']`).

La catégorie (et un lien vers celle-ci) est également ajouté dans les blocs des sujets, afin de rappeler que cette page liste les derniers sujets quelque soit leur catégorie (ils sont ordonnés selon leur date de publication).
@gcodeur

gcodeur approved these changes Jan 30, 2018 edited

C'est tout bon pour moi. Merci et désolé pour le délai !

(Je relance Travis vu que je ne comprends pas l'erreur.)

@gcodeur

This comment has been minimized.

Show comment
Hide comment
@gcodeur

gcodeur Jan 30, 2018

Member

Travis est passé sans problème et les icônes me paraissent adaptées. Si c'est bon pour toi, on peut merger. :)

Member

gcodeur commented Jan 30, 2018

Travis est passé sans problème et les icônes me paraissent adaptées. Si c'est bon pour toi, on peut merger. :)

@rezemika

This comment has been minimized.

Show comment
Hide comment
@rezemika

rezemika Jan 30, 2018

Contributor

C'est tout bon pour moi. Merci et désolé pour le délai !

Aucun soucis ! J'étais moi-même assez occupé ces derniers temps.

Ça marche ! J'enlève le "[WIP]". À la limite on pourra toujours changer les icones plus tard si quelqu'un en suggère d'autres.

Merci ! :)

Contributor

rezemika commented Jan 30, 2018

C'est tout bon pour moi. Merci et désolé pour le délai !

Aucun soucis ! J'étais moi-même assez occupé ces derniers temps.

Ça marche ! J'enlève le "[WIP]". À la limite on pourra toujours changer les icones plus tard si quelqu'un en suggère d'autres.

Merci ! :)

@rezemika rezemika changed the title from [WIP] Ajoute une page avec les derniers sujets publiés to Ajoute une page avec les derniers sujets publiés Jan 30, 2018

@gcodeur gcodeur added this to the Version de développement milestone Jan 30, 2018

@gcodeur gcodeur merged commit a6b460d into zestedesavoir:dev Jan 30, 2018

1 of 2 checks passed

Codacy/PR Quality Review Not so good... This pull request quality could be better.
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details

@rezemika rezemika deleted the rezemika:last-subjects-page branch Jan 30, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment