OpenSourceOrg / dotOrg

Public tracker of opensource.org issues
GNU General Public License v3.0
7 stars 1 forks source link

OpenSource.org | Add last modified on certain pages #9

Closed toncijajic closed 10 months ago

toncijajic commented 10 months ago

To make the visitors aware that some content might not be relevant anymore, we need to add the "Last modified on" to the bottom of page templates.

Scope

We need to add a "Last modified on [date]" (using the_modified_time('F jS, Y');?) to the bottom of the page template. We also need to add "Page created on" to the bottom of the page template.

Page created on 21 July 2019 | Last modified on 19 June 2021

Details

webmink commented 10 months ago

I am concerned that "touches" to pages may make them look current. In general, the date of publication is probably a better indication of relevance than the last modified date (see for example opensource.org/osr). Is it feasible to add both in the case that they differ? I have previously included it as text in the page where it is highly significant.

toncijajic commented 10 months ago

Is it feasible to add both in the case that they differ? I have previously included it as text in the page where it is highly significant.

@webmink good point! Any recommendation on conditional addition in that case - last modified over a year ago OR created over three years ago?

cc @smaffulli for visibility

smaffulli commented 10 months ago

How about we show a "Page created on " with an absolute time: any page created before the migration (February 2023) is a legacy page and we may want to treat those carefully? If all pages before Feb 2023 show a "Page created on" and also show "Last modified on" I think it'd work, until we can review the whole site.

toncijajic commented 10 months ago

If all pages before Feb 2023 show a "Page created on" and also show "Last modified on" I think it'd work, until we can review the whole site.

@smaffulli good idea, works for us!

bernattorras commented 10 months ago

@toncijajic I've pushed these changes into the staging site to confirm if they are being displayed as expected.

Notes / Considerations

smaffulli commented 10 months ago

thanks. I like the idea of moving showing the age of the page below the title, for older pages.

bernattorras commented 10 months ago

I've updated the code to:

Here's an example of the dates in the header.

Screenshots

Page with header image

Page without header image

Let us know if everything is working as expected so we can push the changes into production.

@toncijajic

smaffulli commented 10 months ago

Looks good, thanks!

On Mon, Aug 28, 2023 at 4:54 AM Bernat @.***> wrote:

I've updated the code to:

  • Show the creation and last updated dates below the title by defaultç
  • Show the dates at the end of the post if the page isn't using a header (using the No page header template)

Here's an example of the dates in the header https://open-source-initiative-development.mystagingwebsite.com/osd/. Screenshots

Page with header [image: image] https://user-images.githubusercontent.com/1201868/263692242-b2b57df6-11e0-486f-a816-49b405092be7.png

Page without header [image: image] https://user-images.githubusercontent.com/1201868/263692168-b4f5f9fb-2a28-4f24-ac22-1670339aea75.png

Let us know if everything is working as expected so we can push the changes into production.

@toncijajic https://github.com/toncijajic

— Reply to this email directly, view it on GitHub https://github.com/OpenSourceOrg/dotOrg/issues/9#issuecomment-1695562381, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAKJJAE7OO7XGWMDPLBIAHLXXSBF7ANCNFSM6AAAAAA3YGULLU . You are receiving this because you were mentioned.Message ID: @.***>

toncijajic commented 10 months ago

@bernattorras looks good! Feel free to push to production and let me know. 🙌

bernattorras commented 10 months ago

@toncijajic Done! Feel free to reopen the issue if something else is needed!