Skip to content

fix: add seo / meta tags by front matter#27

Merged
tylerdave merged 6 commits into
mainfrom
seo-meta
Mar 1, 2026
Merged

fix: add seo / meta tags by front matter#27
tylerdave merged 6 commits into
mainfrom
seo-meta

Conversation

@tylerdave
Copy link
Copy Markdown
Member

No description provided.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Mar 1, 2026

Docs Site preview: https://a1969b68.rockgarden.pages.dev

claude[bot]

This comment was marked as outdated.

claude[bot]

This comment was marked as outdated.

claude[bot]

This comment was marked as outdated.

Copy link
Copy Markdown

@claude claude Bot left a comment

Choose a reason for hiding this comment

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

LGTM — all three previously flagged bugs are resolved:

  1. og:url construction — the template now correctly handles index slugs (stripping index and using trailing slash) and clean_urls=False (appending .html), matching the logic in get_url(). The base_url validator already strips trailing slashes, so no double-slash risk.

  2. keywords as YAML list{% if _kw is string %}...{% elif _kw %}...{% endif %} correctly handles both string and list values.

  3. Folder index pages — the {% elif folder is defined %} branch now populates _title, _slug, and _fm from FolderIndex, which has a .frontmatter dict field confirmed in nav/folder_index.py:34.

No new issues found.

@tylerdave tylerdave merged commit 3a05632 into main Mar 1, 2026
3 checks passed
@tylerdave tylerdave deleted the seo-meta branch March 1, 2026 21:35
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.

1 participant