Skip to content

Conversation

@mcanouil
Copy link
Collaborator

@mcanouil mcanouil commented Jan 5, 2026

This pull request fixes a typo. Now the code correctly targets the correct property attribute instead of the name attribute for og:site_name meta tags.

@posit-snyk-bot
Copy link
Collaborator

posit-snyk-bot commented Jan 5, 2026

Snyk checks have passed. No issues have been found so far.

Status Scanner Critical High Medium Low Total (0)
Open Source Security 0 0 0 0 0 issues
Licenses 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

@mcanouil mcanouil changed the title fix: change meta selector from "name" to "property" fix(website-meta.ts): change meta selector from "name" to "property" Jan 5, 2026
@cderv
Copy link
Collaborator

cderv commented Jan 6, 2026

Thanks! Good find !

Can you add a test or a check for this in an existing test for shortcode in title ?

Thanks.

@mcanouil
Copy link
Collaborator Author

mcanouil commented Jan 6, 2026

Can you add a test or a check for this in an existing test for shortcode in title ?

Cannot run any test at the moment.

@cderv cderv merged commit c7b25ea into quarto-dev:main Jan 6, 2026
51 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