Add feed link to post side menu #262

Merged

erik merged 1 commit from feat/task-59bf5b85-post-feed-link into main

2026-07-18 08:01:19 -05:00

erik commented

2026-07-18 07:54:51 -05:00

Owner

Summary

Post-detail sidebars now include the existing Feed navigation card, making /feed directly accessible from article and link pages.

Task

Task: task-59bf5b85

Changes

Add the shared Feed card before the existing article, source, and tag cards on post details.
Cover both article and link sidebar rendering.
Verify selecting the card navigates to /feed in Playwright.

Testing

Unit tests added/updated
npm run test:visual
./scripts/pre-pr.sh

Checklist

./scripts/pre-pr.sh passes
Documentation updated (not needed)
No unrelated changes included

## Summary Post-detail sidebars now include the existing Feed navigation card, making `/feed` directly accessible from article and link pages. ## Task Task: task-59bf5b85 ## Changes - Add the shared Feed card before the existing article, source, and tag cards on post details. - Cover both article and link sidebar rendering. - Verify selecting the card navigates to `/feed` in Playwright. ## Testing - [x] Unit tests added/updated - [x] `npm run test:visual` - [x] `./scripts/pre-pr.sh` ## Checklist - [x] `./scripts/pre-pr.sh` passes - [x] Documentation updated (not needed) - [x] No unrelated changes included

erik added 1 commit

2026-07-18 07:54:51 -05:00

feat: link post sidebars to feed

CI / build-lint-test (pull_request) Successful in 2m18s

Details

4eb78c55fd

Task: #task-59bf5b85

erik commented

2026-07-18 07:57:47 -05:00

Author

Owner

Review summary

The PR reuses the existing Feed sidebar card on both article and link post-detail sidebars and adds direct browser verification of the destination.

Acceptance criteria

Met: Article and link /posts/{slug} sidebars include the shared Feed card with href="/feed", matching existing sidebar styling and ordering conventions.
Met: Playwright selects the Feed card from a post page, verifies the /feed URL, and confirms the Feed heading is rendered.

Blocking issues

None.

Warnings

None.

Verdict

Approved. CI passed for commit 4eb78c55fd, and both acceptance criteria are covered.

### Review summary The PR reuses the existing Feed sidebar card on both article and link post-detail sidebars and adds direct browser verification of the destination. ### Acceptance criteria - **Met:** Article and link `/posts/{slug}` sidebars include the shared Feed card with `href="/feed"`, matching existing sidebar styling and ordering conventions. - **Met:** Playwright selects the Feed card from a post page, verifies the `/feed` URL, and confirms the Feed heading is rendered. ### Blocking issues - None. ### Warnings - None. ### Verdict **Approved.** CI passed for commit `4eb78c55fd`, and both acceptance criteria are covered.

erik merged commit da82c158a8 into main

2026-07-18 08:01:19 -05:00

erik deleted branch feat/task-59bf5b85-post-feed-link

2026-07-18 08:01:19 -05:00

erik referenced this pull request from a commit

2026-07-18 08:01:20 -05:00

Merge pull request 'Add feed link to post side menu' (#262) from feat/task-59bf5b85-post-feed-link into main