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.

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

correction du filtre de validation des tutoriels #2654

Merged
merged 1 commit into from
May 11, 2015
Merged

correction du filtre de validation des tutoriels #2654

merged 1 commit into from
May 11, 2015

Conversation

Eskimon
Copy link
Contributor

@Eskimon Eskimon commented May 10, 2015

Inspiré de #2539

Q R
Correction de bugs ? oui
Nouvelle Fonctionnalité ? non
Tickets (issues) concernés #1653

Cette PR permet de corriger le bug du filtre dans l'interface de validation des tutoriels, qui faisait que malgré qu'on cliquait sur une sous-catégorie, le contenu n'était pas toujours filtré.

Note pour la QA:

  • Créez des sous-categories dans le menu admin (tout en bas dans Utils)
  • Créez au moins deux tutoriels dans 2 catégories différentes et un troisieme qui recoupe une categorie deja prise par un des deux autres
  • Mettez ces tutos en validation
  • Allez dans l'interface de validation de tutos et cliquez sur une catégorie pour fitrer
  • Constatez que seuls les tutos de la catégorie demandée sont afficher.

N'hésitez pas a tester avec des tutoriels qui contiennent plusieurs catégories.

la flemme de perdre mon temps à faire un TU quand on sait que la ZEP-12 le couvrira mieux que moi (plus générique)...

@landscape-bot
Copy link

Code Health
Code quality remained the same when pulling 39ee1e8 on Eskimon:fix-1653 into 25ebb65 on zestedesavoir:dev.

@pierre-24
Copy link
Member

Rapport de QA: OK pour moi, je merge :)

(et effectivement, c'est repris par la ZEP-12, mais pas encore testé unitairement, je l'ai rajouté à la liste)

pierre-24 added a commit that referenced this pull request May 11, 2015
correction du filtre de validation des tutoriels
@pierre-24 pierre-24 merged commit b288aca into zestedesavoir:dev May 11, 2015
@pierre-24 pierre-24 added this to the Version de développement milestone May 11, 2015
@pierre-24 pierre-24 added S-BUG Corrige un problème C-Back Concerne le back-end Django labels May 11, 2015
@Eskimon Eskimon deleted the fix-1653 branch May 15, 2015 09:39
@SpaceFox SpaceFox modified the milestones: Version 15.5.1, Version de développement May 26, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-Back Concerne le back-end Django S-BUG Corrige un problème
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants