Show publication date on petition page #299
Conversation
rsk2
changed the title
…tion date at time of publish
| @@ -330,6 +330,11 @@ <h2 class="text-danger">{% trans "This petition is moderated. You only see it be | |||
| </div> | |||
| </div> | |||
| <div class="presentation"> | |||
| {% if petition.publication_date and petition.show_publication_date %} | |||
| <div class="font-weight-light font-italic"> | |||
| Published on: {{ petition.publication_date|date:"M d, Y" }} | |||
There was a problem hiding this comment.
I would say this date format is specific to english speaking countries and others but do not work for France for instance.
I would say petition.publication_date|date:"SHORT_DATE_FORMAT" would be better, don't you think?
| class Migration(migrations.Migration): | ||
|
|
||
| dependencies = [ | ||
| ('petition', '0016_signature_uses_phone_number_field'), |
There was a problem hiding this comment.
Now there is a migration 0017 (https://github.com/pytition/Pytition/blob/master/pytition/petition/migrations/0017_petitiontemplate_paper_signatures_enabled.py)
Could you "rebase" your PR so that your migrations depend on this one please?
| @@ -0,0 +1,18 @@ | |||
| # Generated by Django 2.2.28 on 2022-12-16 05:04 | |||
There was a problem hiding this comment.
This migration should now depend on 0018 and be renamed as 0019
| @@ -330,6 +330,11 @@ <h2 class="text-danger">{% trans "This petition is moderated. You only see it be | |||
| </div> | |||
| </div> | |||
| <div class="presentation"> | |||
| {% if petition.publication_date and petition.show_publication_date %} | |||
| <div class="font-weight-light font-italic"> | |||
| Published on: {{ petition.publication_date|date:"M d, Y" }} | |||
There was a problem hiding this comment.
Please use text that can be translated via i18n :
{% trans "Published on: "%}
|
Thanks @fallen! I've updated the PR |
#294