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

Tri des sous catégorie par position dans le menu #4827

Merged
merged 2 commits into from Feb 3, 2018

Conversation

Projects
None yet
3 participants
@Anto59290
Contributor

Anto59290 commented Jan 2, 2018

Numéro du ticket concerné (optionnel) : #3615

A la base je voulais juste traiter #3615 à savoir : rajouter un paramètre de position pour les sous catégorie dans le menu. En passant pas la j'ai refacto plusieurs choses qui ne me paraissait pas très logique:

  • forums_pk ne servait pas
  • Les tags exclus étaient gérés à la main, ils sont maintenant gérés via l'ORM
  • Utilisation d'un Q pour la sélection des forums
  • Le cas avec _type non définit ne semble pas utilisé

Contrôle qualité

  • Appliquez la migration
  • Pour l'instant l'ordre des sous catégories n'a pas changé
  • En tant qu'Admin vous pouvez changer la position des sous catégories, pour par exemple mettre les catégories Autres en dernière position.
  • Ce changement impact le menu

Bonne QA :)

Anto59290 added some commits Jan 2, 2018

@coveralls

This comment has been minimized.

Show comment
Hide comment
@coveralls

coveralls Jan 2, 2018

Coverage Status

Coverage increased (+0.03%) to 89.591% when pulling f2075c3 on Anto59290:fix/3615-ulla into 8aebf47 on zestedesavoir:dev.

coveralls commented Jan 2, 2018

Coverage Status

Coverage increased (+0.03%) to 89.591% when pulling f2075c3 on Anto59290:fix/3615-ulla into 8aebf47 on zestedesavoir:dev.

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

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

@artragis artragis merged commit 5eebddc into zestedesavoir:dev Feb 3, 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 3, 2018

Contributor

QA OK

Contributor

artragis commented Feb 3, 2018

QA OK

@artragis artragis added this to the Version de développement milestone Mar 4, 2018

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