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

Display of publication and update dates #1464

Open
Olein-jp opened this issue Apr 7, 2023 · 14 comments
Open

Display of publication and update dates #1464

Olein-jp opened this issue Apr 7, 2023 · 14 comments
Labels
[Component] Learn Theme Website development issues related to the Learn theme. Priority - High High priority issue. [Type] Enhancement New feature request for the Learn website.

Comments

@Olein-jp
Copy link

Olein-jp commented Apr 7, 2023

This is just a request.

I often find Learn content shared on Twitter and other social networking sites, from which I discover content.

However, I cannot quickly see on the page when that content was created, published, or updated, even if it is not new.

So I am convinced that the content would be more meaningful if you included this information somewhere in the first view, if possible, as in the sample.

image

@Olein-jp Olein-jp added Awaiting Triage Issues awaiting triage. See Training Team handbook for how to triage issues. [Content] Feedback Feedback provided about content on Learn. labels Apr 7, 2023
@kaitohm kaitohm added [Type] Enhancement New feature request for the Learn website. and removed [Content] Feedback Feedback provided about content on Learn. labels Apr 10, 2023
@jonathanbossenger
Copy link
Collaborator

Duplicate #1570

@jonathanbossenger
Copy link
Collaborator

Duplicate #1207

@jonathanbossenger jonathanbossenger added Priority - High High priority issue. [Component] Learn Theme Website development issues related to the Learn theme. and removed Awaiting Triage Issues awaiting triage. See Training Team handbook for how to triage issues. labels Jul 13, 2023
@kaitohm
Copy link
Contributor

kaitohm commented Jul 13, 2023

The team discussed the benefits of showing a publication date, and a "last edited" date, on all content on Learn. (Discussion in #1532 ) I propose we add these dates to the sidebar of content:

@jonathanbossenger
Copy link
Collaborator

Assigning this to myself, as this will be the topic of #1788

@psykrotek
Copy link
Contributor

psykrotek commented Aug 23, 2023

I have created a draft pull request, to start work on this issue. I have one question about the dates.

If the updated date and the published date are the same, should they both be shown?

See the example in the screenshot below (tutorials)

7-Tips-to-improve-website-security-learn-wordpress

@jonathanbossenger
Copy link
Collaborator

@Olein-jp @bsanevans I think that if the published date and the updated date are the same, we should only show the published date. That way the updated date only displays if the content has been updated since it was published. This gives a clearer idea to the learner as to the status of the content.

Would you agree?

@Olein-jp
Copy link
Author

@jonathanbossenger I agree!

@jonathanbossenger
Copy link
Collaborator

Thank you @Olein-jp, that was my initial thought, but I wanted to check.

I'm going to go ahead and develop it that way in the meantime.

@Olein-jp
Copy link
Author

@jonathanbossenger The WordPress version corresponding to each tutorial is listed, so it may be possible to guess to some extent when it was created.

However, I found that this would not work for anyone other than those heavily involved with WordPress.

So I think it would be very useful if the date of posting was listed alongside the WordPress version.

Thank you for your lovely correspondence, it is much appreciated.

@jonathanbossenger
Copy link
Collaborator

Noting that the @psykrotek account that created the pull request is a duplicate account I own, I was using it as part of an online workshop on contributing to Learn WordPress with code.

@unprintedch
Copy link

Since the WordPress pace of update is fast I think that we need this to be merge.
And since it is almost ready! Could it be merge?

@jonathanbossenger
Copy link
Collaborator

@unprintedch this is something that I'd love to see merged, but I never got around to finishing.

Perhaps you'd like to create your own PR, include the changes from my original one, and include the addition I've proposed here?

@unprintedch
Copy link

I'll do!

@jonathanbossenger jonathanbossenger removed their assignment Apr 3, 2024
unprintedch added a commit to unprintedch/Learn that referenced this issue Apr 7, 2024
WordPress#1464

Add published/modified date.

Modified date is displayed conditionaly
We use the date to make sure there is at least a day difference.
@ironnysh
Copy link

ironnysh commented Sep 5, 2024

Added some feedback to @unprintedch's PR (#2368).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Component] Learn Theme Website development issues related to the Learn theme. Priority - High High priority issue. [Type] Enhancement New feature request for the Learn website.
Projects
Status: In discussion
Development

No branches or pull requests

6 participants