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

Corrige la gestion des unes #4450

Merged
merged 4 commits into from
Aug 10, 2017
Merged

Conversation

pierre-24
Copy link
Member

@pierre-24 pierre-24 commented Aug 4, 2017

Q R
Type de modification correction de bug / nouvelle fonctionnalité / évolution
Ticket(s) (issue(s)) concerné(s) #4444 #3605

Bonus: plus de formats de date supportés, en ce compris le traditionel d/m/Y H:M (mais d'autres, et l'ancien, fonctionnent toujours)

Note: en fait, régler #3605 est un peu plus tordu que prévu, parce que c'est une fonctionnalité qui n'est supporté aujourd'hui que par chrome (même si les autres suivent). Du coup, ça donne un format de date un peu bizarre pour ceux qui utilisent un navigateur moins récent.

QA

  • Vérifier que mettre une date invalide à la création d'une une ne fait plus une 500
  • Vérfier qu'il est possible de changer la date d'une une à postériori
  • Code review

@pierre-24 pierre-24 added C-Back Concerne le back-end Django S-BUG Corrige un problème labels Aug 4, 2017
@coveralls
Copy link

coveralls commented Aug 4, 2017

Coverage Status

Coverage increased (+0.08%) to 89.339% when pulling 2ba83b0 on pierre-24:fix_featured_items into b344d36 on zestedesavoir:dev.

@vhf vhf added the QA svp label Aug 5, 2017
Copy link
Contributor

@motet-a motet-a left a comment

Choose a reason for hiding this comment

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

LGTM mais j’ai pas lu les sources.

@abdelaz3r
Copy link
Contributor

NOK : dans certains cas de modifications la une est dupliquée

@coveralls
Copy link

coveralls commented Aug 6, 2017

Coverage Status

Coverage decreased (-0.2%) to 89.059% when pulling 90305c6 on pierre-24:fix_featured_items into b344d36 on zestedesavoir:dev.

@coveralls
Copy link

coveralls commented Aug 9, 2017

Coverage Status

Coverage increased (+0.2%) to 89.026% when pulling 7930ab5 on pierre-24:fix_featured_items into 98854ee on zestedesavoir:dev.

@@ -45,6 +46,7 @@
class="topic-image"
>
<span class="topic-title">{{ featured_resource.title }}</span>
<span class="topic-subtitle">{% trans 'Publiée' %} {{ featured_resource.pubdate|format_date:True|lower }}</span>
Copy link
Contributor

Choose a reason for hiding this comment

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

Je verrais mieux ça avec des retours à la ligne :

<span class="topic-subtitle">
    {% trans 'Publiée' %}
    {{ featured_resource.pubdate|format_date:True|lower }}
</span>

@vhf vhf merged commit 606891b into zestedesavoir:dev Aug 10, 2017
@gllmc gllmc removed the QA svp label Aug 12, 2017
sandhose pushed a commit that referenced this pull request Sep 18, 2017
* Corrige la gestion des unes (fix #4444)

* CBV that!

* Ajoute un input type="datetime-local"
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.

6 participants