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

1464---Adding-publish-and-updated-dates-dc #2368

Open
wants to merge 4 commits into
base: trunk
Choose a base branch
from

Conversation

unprintedch
Copy link

#1464

Add published/modified date.

Modified date is displayed conditionally
We use the date to make sure there is at least a day difference.

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.
@unprintedch
Copy link
Author

unprintedch commented Apr 7, 2024

I did not use the nice get_post_timestamp to get the timestamps since it compares the date and time, and it is better to display only if there is a "day difference".
It compares with "!=" since the modified date must be after...
This is not DRY, we could make a function since it repeats in two places.

@devmuhib009 devmuhib009 added [Component] Learn Plugin Website development issues related to the Learn plugin. [Dev] Needs Review Pull request needing a review. labels Apr 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Component] Learn Plugin Website development issues related to the Learn plugin. [Dev] Needs Review Pull request needing a review.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants