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

Add support for optional updateDate #203

Open
martinstabe opened this issue Jul 30, 2018 · 1 comment
Open

Add support for optional updateDate #203

martinstabe opened this issue Jul 30, 2018 · 1 comment
Labels
V2 v2-related issue

Comments

@martinstabe
Copy link
Member

martinstabe commented Jul 30, 2018

Standard FT stories display PublishDate unless the UpdateTimestamp flag is set to true.

Default starter kit behaviour is currently the reverse: displaying the latest build date unless a publication date is fixed.

This should be possible to fix by:

  1. Require existing publishedDate to be a static date (default could be the scaffolding date)

  2. adding two new fields to the config: An UpdateTimestamp Boolean and an optional date UpdateDate which defaults to the current time.

  3. adding a condition to views/includes/article-head.html that show "Updated" before the date
    and uses the UpdateDate if UpdateTimestamp is true.

@martinstabe martinstabe changed the title Add config field for updateDate Add support for optional updateDate Jul 30, 2018
@joannaskao
Copy link
Member

Do you have an example showing how FT.com displays "updated" on updated articles? When we spoke about this about a year ago, I was told that since the site doesn't do this, we shouldn't either.

@aendra-rininsland aendra-rininsland added the V2 v2-related issue label Jan 15, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
V2 v2-related issue
Projects
None yet
Development

No branches or pull requests

3 participants