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

Liens vers sous-catégories (fix #4507) #4508

Merged
merged 1 commit into from Aug 18, 2017

Conversation

Projects
None yet
4 participants
@pierre-24
Member

pierre-24 commented Aug 11, 2017

Q R
Type de modification correction de bug
Ticket(s) (issue(s)) concerné(s) #4507

QA

  • Vérifier que tout les liens vers des (sous)-catégories fonctionnent, en particulier sur les tutos/articles et les items dans les listes, mais aussi ailleurs si j'en ai oublié.
  • Code review
@coveralls

This comment has been minimized.

Show comment
Hide comment
@coveralls

coveralls Aug 11, 2017

Coverage Status

Coverage decreased (-15.9%) to 73.412% when pulling 75fbf6b on pierre-24:fix_4507 into ab5fc57 on zestedesavoir:dev.

coveralls commented Aug 11, 2017

Coverage Status

Coverage decreased (-15.9%) to 73.412% when pulling 75fbf6b on pierre-24:fix_4507 into ab5fc57 on zestedesavoir:dev.

@coveralls

This comment has been minimized.

Show comment
Hide comment
@coveralls

coveralls Aug 11, 2017

Coverage Status

Coverage remained the same at 89.347% when pulling 75fbf6b on pierre-24:fix_4507 into ab5fc57 on zestedesavoir:dev.

coveralls commented Aug 11, 2017

Coverage Status

Coverage remained the same at 89.347% when pulling 75fbf6b on pierre-24:fix_4507 into ab5fc57 on zestedesavoir:dev.

@pierre-24 pierre-24 added the QA svp label Aug 11, 2017

@pierre-24 pierre-24 changed the title from Liens vers sous-catégories (fix #4507) to [v25] Liens vers sous-catégories (fix #4507) Aug 11, 2017

@pierre-24 pierre-24 changed the title from [v25] Liens vers sous-catégories (fix #4507) to Liens vers sous-catégories (fix #4507) Aug 11, 2017

@motet-a

On peut pas mettre des retours à la ligne ? Ces lignes sont vraiment très longues :-p

@@ -71,7 +71,7 @@
</span>
<ul>
{% for category in content.subcategory.all %}
{% if forloop.first %}{% elif forloop.last %} {% trans "et" %}{% else %},{% endif %} <a href="{% url 'publication:list' %}?category={{ category.slug }}">{{ category.title }}</a>
{% if forloop.first %}{% elif forloop.last %} {% trans "et" %}{% else %},{% endif %} <a href="{% url 'publication:list' %}?subcategory={{ category.slug }}">{{ category.title }}</a>

This comment has been minimized.

@motet-a

motet-a Aug 11, 2017

Member

Bon, je connais encore bien mal ce langage de templates mais pourquoi ne pas mettre :

{% if forloop.last %}...

Au lieu de :

{% if forloop.first %}{% elif forloop.last %}...
@motet-a

motet-a Aug 11, 2017

Member

Bon, je connais encore bien mal ce langage de templates mais pourquoi ne pas mettre :

{% if forloop.last %}...

Au lieu de :

{% if forloop.first %}{% elif forloop.last %}...

This comment has been minimized.

@artragis

artragis Aug 17, 2017

Contributor

pas si sûr motet-a, le sens de cette conditiion est plutôt

{% if forloop.last and not forloop.first %}

@artragis

artragis Aug 17, 2017

Contributor

pas si sûr motet-a, le sens de cette conditiion est plutôt

{% if forloop.last and not forloop.first %}

This comment has been minimized.

@motet-a

motet-a Aug 17, 2017

Member

Ah OK, merci !

@motet-a

motet-a Aug 17, 2017

Member

Ah OK, merci !

@artragis

This comment has been minimized.

Show comment
Hide comment
@artragis

artragis Aug 17, 2017

Contributor

@pierre-24 peux-tu ajouter quelques petits retours à la lignes pour raccourcir la largeur des templates? Je fais la QA juste après ça.

Contributor

artragis commented Aug 17, 2017

@pierre-24 peux-tu ajouter quelques petits retours à la lignes pour raccourcir la largeur des templates? Je fais la QA juste après ça.

@pierre-24

This comment has been minimized.

Show comment
Hide comment
@pierre-24

pierre-24 Aug 18, 2017

Member

@artragis : je viens de me souvenir pourquoi il n'y a pas de retours à la ligne:

screenshot from 2017-08-18 09 26 37

(placement de la virgule)

Z'avez gagné un rebase.

Member

pierre-24 commented Aug 18, 2017

@artragis : je viens de me souvenir pourquoi il n'y a pas de retours à la ligne:

screenshot from 2017-08-18 09 26 37

(placement de la virgule)

Z'avez gagné un rebase.

@coveralls

This comment has been minimized.

Show comment
Hide comment
@coveralls

coveralls Aug 18, 2017

Coverage Status

Coverage remained the same at 89.377% when pulling 214f28d on pierre-24:fix_4507 into 90f4f57 on zestedesavoir:dev.

coveralls commented Aug 18, 2017

Coverage Status

Coverage remained the same at 89.377% when pulling 214f28d on pierre-24:fix_4507 into 90f4f57 on zestedesavoir:dev.

@artragis artragis merged commit 3a1ac20 into zestedesavoir:dev Aug 18, 2017

2 checks passed

codacy/pr 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 Aug 18, 2017

Contributor

OK

Contributor

artragis commented Aug 18, 2017

OK

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