Add articles page like reference site #160

Merged
erik merged 1 commit from feat/task-2356da49-articles-page into main 2026-06-19 22:31:32 -05:00
Owner

Summary

  • Add public /articles page modeled on the erikcraddock.me articles route.
  • List only published article posts, excluding notes, links, and drafts.
  • Render reference-style responsive article cards with preview images, metadata, tags, excerpts, and read-more links.
  • Add article pagination with invalid/out-of-range page redirects.
  • Update article navigation links from homepage/feed sidebar to /articles.
  • Add tests for article filtering, page rendering, preview images, tags, pagination, and redirects.

Verification

  • npm run build --workspace @slugkit/template-site
  • npm test --workspace @slugkit/template-site -- src/routes/__tests__/public.test.ts
  • ./scripts/pre-pr.sh
  • Browser smoke test for /articles
## Summary - Add public `/articles` page modeled on the `erikcraddock.me` articles route. - List only published article posts, excluding notes, links, and drafts. - Render reference-style responsive article cards with preview images, metadata, tags, excerpts, and read-more links. - Add article pagination with invalid/out-of-range page redirects. - Update article navigation links from homepage/feed sidebar to `/articles`. - Add tests for article filtering, page rendering, preview images, tags, pagination, and redirects. ## Verification - `npm run build --workspace @slugkit/template-site` - `npm test --workspace @slugkit/template-site -- src/routes/__tests__/public.test.ts` - `./scripts/pre-pr.sh` - Browser smoke test for `/articles`
feat: add reference-style articles page
All checks were successful
CI / build-lint-test (pull_request) Successful in 45s
2b940f4958
Task: #task-2356da49
Author
Owner

PR Review: Approved

Summary

The PR adds /articles modeled on the erikcraddock.me articles page: a responsive article-card grid, preview images from Markdown content, metadata/tags/excerpts/read-more links, and page-query pagination. It also updates existing article navigation links to point at /articles.

Acceptance criteria

  • GET /articles renders a public articles page — implemented and browser-smoke tested.
  • The page lists published articles and excludes notes and links — covered by route test with mixed published content and a draft article.
  • The page follows the reference site's articles page pattern while matching Slugkit conventions — uses existing Tailwind card styling and ArticleCard conventions.
  • Existing public routes such as /, /posts/:slug, /tags, and /feed.xml still work — full pre-PR suite passes.
  • Relevant tests and local checks pass.

Blocking issues

None.

Warnings

None.

Verification

  • npm run build --workspace @slugkit/template-site
  • npm test --workspace @slugkit/template-site -- src/routes/__tests__/public.test.ts
  • ./scripts/pre-pr.sh
  • Browser smoke test for /articles
  • Forgejo Actions run #222 passed.

Verdict

Approved. Stop at the human merge gate.

PR Review Worker Result

  • pr: open
  • review: approved
  • pr_comment: posted
  • task_comment: posted
  • blocking_issues: 0
  • warnings: 0
  • criteria_total: 5
  • criteria_met: 5
  • criteria_missing: 0
## PR Review: Approved ### Summary The PR adds `/articles` modeled on the `erikcraddock.me` articles page: a responsive article-card grid, preview images from Markdown content, metadata/tags/excerpts/read-more links, and page-query pagination. It also updates existing article navigation links to point at `/articles`. ### Acceptance criteria - [x] `GET /articles` renders a public articles page — implemented and browser-smoke tested. - [x] The page lists published articles and excludes notes and links — covered by route test with mixed published content and a draft article. - [x] The page follows the reference site's articles page pattern while matching Slugkit conventions — uses existing Tailwind card styling and `ArticleCard` conventions. - [x] Existing public routes such as `/`, `/posts/:slug`, `/tags`, and `/feed.xml` still work — full pre-PR suite passes. - [x] Relevant tests and local checks pass. ### Blocking issues None. ### Warnings None. ### Verification - `npm run build --workspace @slugkit/template-site` - `npm test --workspace @slugkit/template-site -- src/routes/__tests__/public.test.ts` - `./scripts/pre-pr.sh` - Browser smoke test for `/articles` - Forgejo Actions run #222 passed. ### Verdict Approved. Stop at the human merge gate. PR Review Worker Result - pr: open - review: approved - pr_comment: posted - task_comment: posted - blocking_issues: 0 - warnings: 0 - criteria_total: 5 - criteria_met: 5 - criteria_missing: 0
erik merged commit 83cdfce5ce into main 2026-06-19 22:31:32 -05:00
erik deleted branch feat/task-2356da49-articles-page 2026-06-19 22:31:32 -05:00
Sign in to join this conversation.
No description provided.