Skip to content

Avoid printing og:description if empty #42

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

Merged
merged 1 commit into from
May 26, 2021
Merged

Avoid printing og:description if empty #42

merged 1 commit into from
May 26, 2021

Conversation

eguidotti
Copy link

This is a minimal PR to check if og:description is empty and avoid printing an empty tag.

@eguidotti eguidotti mentioned this pull request May 26, 2021
@TheTripleV
Copy link
Collaborator

If this is regarding #41, description is auto-generated from the contents of each page. Unless a page is empty, bool(description) will always be true. This is a good check to have though. Thanks.

@eguidotti
Copy link
Author

Nice! I guess this would also avoid printing the description when the description length is set to zero (besides empty pages). Do you already have some plans about a possible merging date? Thanks a lot!

@TheTripleV TheTripleV merged commit 16a291d into sphinx-doc:main May 26, 2021
@TheTripleV
Copy link
Collaborator

TheTripleV commented May 26, 2021

A new release has been made and published to PyPi.

@eguidotti
Copy link
Author

That's great, I look forward to trying it out!

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Apr 5, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants