Skip to content

Add a article:modified_time property for articles#515

Open
CookiePLMonster wants to merge 2 commits intojekyll:masterfrom
CookiePLMonster:master
Open

Add a article:modified_time property for articles#515
CookiePLMonster wants to merge 2 commits intojekyll:masterfrom
CookiePLMonster:master

Conversation

@CookiePLMonster
Copy link
Contributor

@CookiePLMonster CookiePLMonster commented May 2, 2025

This PR adds support for article:modified_time meta property, utilizing the same seo_tag drop values as JSON-LD does.

Two other PRs adding this feature (#447 and #505) are also open, but my attempt has the following advantages:

  1. seo_tag drop values are used for complete consistency with the JSON-LD output. This ensures compatibility with bothpage.last_modified_at and seo.date_modified front matter attributes. I also extended this to the published date instead of using page.date directly.
  2. Updated the Advanced Usage docs to mention that seo.date_modified also influences this new meta property.

Verified working using my own website - on a test blog post, I can see a new meta property added:

<meta property="og:type" content="article" />
<meta property="article:published_time" content="2025-04-23T15:30:00+02:00" />
<meta property="article:modified_time" content="2025-04-23T15:30:00+02:00" />

The behaviour of adding the modification time even if it equals the publishing time matches jekyll-feed.

@CookiePLMonster
Copy link
Contributor Author

CookiePLMonster commented May 2, 2025

@ashmaroli I created this PR in response to our discussion in #505 - whichever PR ends up being merged, the other one (as well as #447) can be closed.

@CookiePLMonster CookiePLMonster force-pushed the master branch 2 times, most recently from 615bd64 to c1b77e3 Compare May 4, 2025 13:19
@CookiePLMonster
Copy link
Contributor Author

Are there any blockers preventing this from moving forward?

@mattr-
Copy link
Member

mattr- commented Jan 12, 2026

Time, energy, and attention. Jekyll does not have any full time maintainers and we all have other demands on our time that are keeping most things from moving forward at the moment. Your patience is appreciated. ❤️

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants