Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Corrige filtres, ordre et nombre de contenus #4484

Merged
merged 12 commits into from
Aug 10, 2017
Merged

Corrige filtres, ordre et nombre de contenus #4484

merged 12 commits into from
Aug 10, 2017

Conversation

pierre-24
Copy link
Member

@pierre-24 pierre-24 commented Aug 6, 2017 •
edited
Loading

Q R
Type de modification correction de bug
Ticket(s) (issue(s)) concerné(s) #4473 #4476 #4481

QA

  • Publier un contenu avec deux sous-catégories (et un tag). Vérifier qu'un seul contenu est compté dans la biblithèque (et dans la (sous-)catégorie correspondante)
  • Publier un second contenu avec deux sous-catégories et de même type que le premier, vérifier sur la page de niveau 4 que c'est bien ce second contenu qui est affiché premier. Vérifier qu'il y a bien 2 contenus compté partout.
  • Changer le titre du second contenu (de manière à ce que le slug) change. Vérifier qu'il y a toujours bien 2 contenus partout.
  • Vérifier que le clic sur un tag fonctionne bien.
  • Code review

@pierre-24 pierre-24 changed the title Corrige ordre et nombre de contenus [WIP] Corrige ordre et nombre de contenus Aug 6, 2017
@coveralls
Copy link

coveralls commented Aug 6, 2017 •
edited
Loading

Coverage Status

Coverage increased (+0.02%) to 88.899% when pulling 940e8b1 on pierre-24:fix_order_contents into 482960a on zestedesavoir:dev.

@pierre-24 pierre-24 mentioned this pull request Aug 7, 2017
Closed
@pierre-24 pierre-24 changed the title [WIP] Corrige ordre et nombre de contenus [WIP] Corrige filtres, ordre et nombre de contenus Aug 7, 2017
@pierre-24 pierre-24 changed the title [WIP] Corrige filtres, ordre et nombre de contenus [v25] Corrige filtres, ordre et nombre de contenus Aug 7, 2017
@pierre-24 pierre-24 added C-Back Concerne le back-end Django C-Front Concerne l'interface du site S-Régression Corrige un problème sur un composant qui fonctionnait auparavant QA svp labels Aug 7, 2017
@coveralls
Copy link

coveralls commented Aug 7, 2017 •
edited
Loading

Coverage Status

Coverage increased (+0.3%) to 89.199% when pulling 257c5e1 on pierre-24:fix_order_contents into 482960a on zestedesavoir:dev.

@coveralls
Copy link

coveralls commented Aug 7, 2017 •
edited
Loading

Coverage Status

Coverage increased (+0.3%) to 89.199% when pulling 669611d on pierre-24:fix_order_contents into 34204d5 on zestedesavoir:dev.

@coveralls
Copy link

coveralls commented Aug 8, 2017 •
edited
Loading

Coverage Status

Coverage increased (+0.3%) to 89.174% when pulling 94aae46 on pierre-24:fix_order_contents into d09f099 on zestedesavoir:dev.

@coveralls
Copy link

coveralls commented Aug 8, 2017 •
edited
Loading

Coverage Status

Coverage increased (+0.3%) to 89.174% when pulling 0f90050 on pierre-24:fix_order_contents into d09f099 on zestedesavoir:dev.

@coveralls
Copy link

coveralls commented Aug 8, 2017 •
edited
Loading

Coverage Status

Coverage increased (+0.3%) to 89.174% when pulling bf0321d on pierre-24:fix_order_contents into d09f099 on zestedesavoir:dev.

@coveralls
Copy link

Coverage Status

Coverage decreased (-33.1%) to 55.746% when pulling 8e08848 on pierre-24:fix_order_contents into 98854ee on zestedesavoir:dev.

@coveralls
Copy link

coveralls commented Aug 9, 2017 •
edited
Loading

Coverage Status

Coverage increased (+0.3%) to 89.175% when pulling 8e08848 on pierre-24:fix_order_contents into 98854ee on zestedesavoir:dev.

motet-a
motet-a reviewed Aug 9, 2017
View reviewed changes
zds/utils/factories.py
class Meta:
model = Category

title = factory.Sequence('Ma catégorie No{0}'.format)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Remplacer par 'Ma catégorie nº {0}' ?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

En fait, a cet endroit là, c'est du pur détail, parce que c'est pour les tests :)

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah OK :-)

@vhf vhf merged commit 1a9fa1d into zestedesavoir:dev Aug 10, 2017
@vhf vhf added this to the Version de développement milestone Aug 10, 2017
@gllmc gllmc removed the QA svp label Aug 12, 2017
@vhf vhf changed the title [v25] Corrige filtres, ordre et nombre de contenus Corrige filtres, ordre et nombre de contenus Aug 20, 2017
sandhose pushed a commit that referenced this pull request Sep 18, 2017
@pierre-24 @sandhose
[v25] Corrige filtres, ordre et nombre de contenus (#4484)
1a62c12 
* Corrige ordre et nombre de contenus

* Distinct and Unit Test

* Fini le filtre par tag

* fix tests

* utilise settins.ZDS_APP pour le nombre de contenus

* not so fast

* C'est 4!

* remarques de @vhf

* Vérifie que la sous-catégorie appartient à la catégorie
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@motet-a motet-a motet-a left review comments

Assignees
No one assigned
Labels
C-Back Concerne le back-end Django C-Front Concerne l'interface du site S-Régression Corrige un problème sur un composant qui fonctionnait auparavant
Projects
None yet
Milestone
v25
Development

Successfully merging this pull request may close these issues.
5 participants
@pierre-24 @coveralls @motet-a @vhf @gllmc