Skip to content

fix(theme): entry links follow collection permalinks; auto-merge waits for checks - #19

Merged
Madraka merged 1 commit into
mainfrom
theme/entry-permalinks
Oct 22, 2026
Merged

Madraka merged 1 commit into
mainfrom
theme/entry-permalinks

Conversation

@Madraka

@Madraka Madraka commented Oct 22, 2026

Copy link
Copy Markdown
Contributor
  • Theme: EntryList uses entryPath(media?.permalinks, e), and [...path].astro passes permalinks: site in the media context (plus site to loadComments), matching the starter since Cloudhal 0.3.11.
  • cloudhal-update.yml: gh pr merge --auto. Dependabot chore: Bump the cloudhal group with 12 updates #18's merge ran before the required Workers Builds check finished and was refused by the branch policy; with --auto it waits.

Checked locally with cloudhal dev: / and /blog render (221 kB, 205 kB), entry links are /blog/<slug>, no errors in the log.

🤖 Generated with Claude Code

…s for checks

- `EntryList` calls `entryPath(media?.permalinks, e)` and the page passes
  `permalinks: site` (and `site` to `loadComments`), as the starter does
  since Cloudhal 0.3.11. The old one-argument call works again in 0.3.13
  but ignores collection permalinks.
- `cloudhal-update.yml` merges with `--auto`: the merge ran before the
  required Workers Builds check had finished and was refused (#18).

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Signed-off-by: Madraka <8898640+Madraka@users.noreply.github.com>
@Madraka
Madraka merged commit 1a125d2 into main Oct 22, 2026
3 checks passed
@Madraka
Madraka deleted the theme/entry-permalinks branch October 22, 2026 15:54
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