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

Barre de navigation pour parcourir les articles et les tribunes en haut et en bas #4795

Merged
merged 2 commits into from Feb 2, 2018

Conversation

Projects
None yet
6 participants
@Situphen
Contributor

Situphen commented Nov 14, 2017

Ajoute une barre de navigation en haut et en bas pour les articles et les billets. Quand il s'agit de billets, la navigation se fait seulement entre les billets du même auteur.

(Corrige #4599 "Barre de navigation pour parcourir les articles en haut")

QA :

  • Aller sur n'importe quel article, et vérifier que la barre de navigation permet de naviguer parmi tous les articles
  • Aller sur un billet dont l'auteur a publié un seul billet, et vérifier qu'il n'y a pas de barre de navigation
  • Aller sur un billet dont l'auteur a publié plusieurs billets, et vérifier que la barre de navigation permet de naviguer seulement parmi les billets de l'auteur

Quand il y a plusieurs auteurs, la navigation se fait parmi un mélange de leurs billets

@Situphen Situphen added the C-Front label Nov 14, 2017

@zestedesavoir zestedesavoir deleted a comment from coveralls Nov 15, 2017

@zestedesavoir zestedesavoir deleted a comment from coveralls Nov 15, 2017

@zestedesavoir zestedesavoir deleted a comment from coveralls Nov 18, 2017

@zestedesavoir zestedesavoir deleted a comment from coveralls Nov 18, 2017

@zestedesavoir zestedesavoir deleted a comment from coveralls Nov 18, 2017

@zestedesavoir zestedesavoir deleted a comment from coveralls Nov 18, 2017

@Situphen Situphen added the C-Back label Dec 2, 2017

@artragis artragis added this to the Version de développement milestone Dec 5, 2017

@gcodeur

LGTM

@zestedesavoir zestedesavoir deleted a comment from coveralls Dec 10, 2017

@Anto59290

Quelques commentaire de clareté

Show outdated Hide outdated zds/tutorialv2/views/published.py Outdated
Show outdated Hide outdated zds/tutorialv2/views/published.py Outdated
Ajoute une barre de navigation pour les billets (et corrige #4599 "Ba…
…rre de navigation pour parcourir les articles en haut")
@Situphen

This comment has been minimized.

Show comment
Hide comment
@Situphen

Situphen Dec 19, 2017

Contributor

J'ai ajouté le code ultra optimisé de @Anto59290 (encore merci à toi) !

Pour les articles j'ai l'impression que la pagination fonctionne mal quand je compare avec la liste des articles de la page d'accueil mais c'est probablement un faux positif lié aux dates de publication de ma base de donnée (les articles sont générés par la commande python manage.py load_fixtures et ont donc une date de publication très proche de l'ordre de la seconde) car les tests passent.

Contributor

Situphen commented Dec 19, 2017

J'ai ajouté le code ultra optimisé de @Anto59290 (encore merci à toi) !

Pour les articles j'ai l'impression que la pagination fonctionne mal quand je compare avec la liste des articles de la page d'accueil mais c'est probablement un faux positif lié aux dates de publication de ma base de donnée (les articles sont générés par la commande python manage.py load_fixtures et ont donc une date de publication très proche de l'ordre de la seconde) car les tests passent.

@artragis

This comment has been minimized.

Show comment
Hide comment
@artragis

artragis Dec 19, 2017

Contributor

plus qu'à gérer pep8 et ça sera impec

Contributor

artragis commented Dec 19, 2017

plus qu'à gérer pep8 et ça sera impec

@Situphen

This comment has been minimized.

Show comment
Hide comment
@Situphen

Situphen Dec 19, 2017

Contributor

Corrigé ! J'ai remis le Git Hook de pre-commit pour ne plus que ça m'arrive :D

Contributor

Situphen commented Dec 19, 2017

Corrigé ! J'ai remis le Git Hook de pre-commit pour ne plus que ça m'arrive :D

@coveralls

This comment has been minimized.

Show comment
Hide comment
@coveralls

coveralls Dec 19, 2017

Coverage Status

Coverage increased (+0.008%) to 89.539% when pulling 3eec2f2 on Situphen:fix-4599 into 35c46e8 on zestedesavoir:dev.

coveralls commented Dec 19, 2017

Coverage Status

Coverage increased (+0.008%) to 89.539% when pulling 3eec2f2 on Situphen:fix-4599 into 35c46e8 on zestedesavoir:dev.

@zestedesavoir zestedesavoir deleted a comment from coveralls Dec 19, 2017

@zestedesavoir zestedesavoir deleted a comment from coveralls Dec 19, 2017

@zestedesavoir zestedesavoir deleted a comment from coveralls Dec 19, 2017

@zestedesavoir zestedesavoir deleted a comment from coveralls Dec 19, 2017

@artragis

This comment has been minimized.

Show comment
Hide comment
@artragis

artragis Dec 20, 2017

Contributor

plus qu'à faire la QA du coup.

Contributor

artragis commented Dec 20, 2017

plus qu'à faire la QA du coup.

@artragis artragis merged commit 9d39735 into zestedesavoir:dev Feb 2, 2018

2 checks passed

Codacy/PR Quality Review Good work! A positive pull request.
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@artragis

This comment has been minimized.

Show comment
Hide comment
@artragis

artragis Feb 2, 2018

Contributor

Je merge car gcodeur avait dit "lgtm"

Contributor

artragis commented Feb 2, 2018

Je merge car gcodeur avait dit "lgtm"

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