Skip to content

Enhance OpenGraph tags#78

Merged
MaevaBrunelles merged 5 commits intopelican-plugins:mainfrom
patrickfournier:sitename-and-summary
Nov 26, 2025
Merged

Enhance OpenGraph tags#78
MaevaBrunelles merged 5 commits intopelican-plugins:mainfrom
patrickfournier:sitename-and-summary

Conversation

@patrickfournier
Copy link
Contributor

  • Add site name tag
  • Description tag falls back to a plain text version of the article summary.

@patrickfournier
Copy link
Contributor Author

patrickfournier commented Nov 18, 2025

This PR should be merged after PR #77

@patrickfournier patrickfournier marked this pull request as ready for review November 21, 2025 00:29
Copy link
Contributor

@justinmayer justinmayer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for your work on this, Patrick! 💫

Copy link
Collaborator

@MaevaBrunelles MaevaBrunelles left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is it possible to add or update a test that assert the fallback to the summary description in case of no description nor og_description?

@patrickfournier
Copy link
Contributor Author

patrickfournier commented Nov 25, 2025

Sure. Test added. Sorry for the messy history.

Copy link
Collaborator

@MaevaBrunelles MaevaBrunelles left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All good for me, thanks!

@MaevaBrunelles MaevaBrunelles merged commit 7abb813 into pelican-plugins:main Nov 26, 2025
7 checks passed
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.

3 participants