erik/slugkit

Fork 0

Update sources page like reference site #150

New issue

Closed

opened 2026-06-19 18:49:13 -05:00 by erik · 3 comments

erik commented

2026-06-19 18:49:13 -05:00

Owner

Goal

Model the Slugkit public sources page after the sources page in ~/Private/code/github/evcraddock/erikcraddock.me/.

Requirements

Use the erikcraddock.me sources route as the reference, especially src/routes/pages.tsx.
Update Slugkit's public /sources route to follow the referenced sources page structure and visual treatment.
Preserve the source detail route at /sources/:id and adapt it only if needed for consistency.
Use Slugkit's existing source, contact, account, template, and CSS conventions.
Show useful source metadata such as name, URL, description, contacts, and accounts where available.
Add or update tests for the rendered sources page where practical.

Acceptance criteria

GET /sources renders a public sources page modeled after the reference site.
The page lists Slugkit sources with useful summary metadata.
GET /sources/:id still works for source details.
Existing public routes such as /, /posts/:slug, /feed, /articles, and /tags still work.
Relevant tests and local checks pass.

Dependencies

None

## Goal Model the Slugkit public sources page after the sources page in `~/Private/code/github/evcraddock/erikcraddock.me/`. ## Requirements - Use the `erikcraddock.me` sources route as the reference, especially `src/routes/pages.tsx`. - Update Slugkit's public `/sources` route to follow the referenced sources page structure and visual treatment. - Preserve the source detail route at `/sources/:id` and adapt it only if needed for consistency. - Use Slugkit's existing source, contact, account, template, and CSS conventions. - Show useful source metadata such as name, URL, description, contacts, and accounts where available. - Add or update tests for the rendered sources page where practical. ## Acceptance criteria - [ ] `GET /sources` renders a public sources page modeled after the reference site. - [ ] The page lists Slugkit sources with useful summary metadata. - [ ] `GET /sources/:id` still works for source details. - [ ] Existing public routes such as `/`, `/posts/:slug`, `/feed`, `/articles`, and `/tags` still work. - [ ] Relevant tests and local checks pass. ## Dependencies - None

erik added the

labels

2026-06-19 18:49:13 -05:00

erik added

status:inprogress

and removed

status:active

labels

2026-06-19 21:44:41 -05:00

erik commented

2026-06-19 21:44:41 -05:00

Author

Owner

Synced from todu comment by @todu on 2026-06-20T02:40:00.221Z

Update

Started implementation on branch feat/task-a17eea31-sources-reference-page.
Plan: model /sources after the reference Recommended Sites page, preserve /sources/:id, update tests, and keep existing public routes working.

_Synced from todu comment by @todu on 2026-06-20T02:40:00.221Z_ ### Update - Started implementation on branch `feat/task-a17eea31-sources-reference-page`. - Plan: model `/sources` after the reference Recommended Sites page, preserve `/sources/:id`, update tests, and keep existing public routes working.

erik

2026-06-19 21:50:12 -05:00

closed this issue
added
status:done
and removed
status:inprogress
labels

erik commented

2026-06-19 21:50:12 -05:00

Author

Owner

Synced from todu comment by @todu on 2026-06-20T02:49:00.021Z

Closing summary

All acceptance criteria are met.

GET /sources now renders a reference-style Recommended Sites page with search, live result updates, result counts, no-result states, and responsive source cards.
Source cards show useful metadata: name, hostname/URL, description, contacts, and accounts where available.
GET /sources/:id still works and now uses a consistent reference-style detail layout.
Existing public routes were covered by the full pre-PR check suite.
Verification passed: npm run build, ./scripts/pre-pr.sh, browser smoke testing for /sources and /sources/:id, and Forgejo Actions run #217.

PR #158 was merged into main after explicit human approval.

_Synced from todu comment by @todu on 2026-06-20T02:49:00.021Z_ ### Closing summary All acceptance criteria are met. - `GET /sources` now renders a reference-style Recommended Sites page with search, live result updates, result counts, no-result states, and responsive source cards. - Source cards show useful metadata: name, hostname/URL, description, contacts, and accounts where available. - `GET /sources/:id` still works and now uses a consistent reference-style detail layout. - Existing public routes were covered by the full pre-PR check suite. - Verification passed: `npm run build`, `./scripts/pre-pr.sh`, browser smoke testing for `/sources` and `/sources/:id`, and Forgejo Actions run #217. PR #158 was merged into `main` after explicit human approval.

erik commented

2026-06-19 21:50:12 -05:00

Author

Owner

Synced from todu comment by @todu on 2026-06-20T02:45:20.023Z

Review update

PR: #158 is open and reviewed.
Result: approved; no blocking issues or warnings.
Verified: npm run build, ./scripts/pre-pr.sh, Forgejo Actions run #217, and browser smoke testing for /sources and /sources/:id.
Acceptance: 5/5 criteria met.
Follow-up: waiting for explicit human merge approval.

_Synced from todu comment by @todu on 2026-06-20T02:45:20.023Z_ ### Review update - PR: https://forge.caradoc.com/erik/slugkit/pulls/158 is open and reviewed. - Result: approved; no blocking issues or warnings. - Verified: `npm run build`, `./scripts/pre-pr.sh`, Forgejo Actions run #217, and browser smoke testing for `/sources` and `/sources/:id`. - Acceptance: 5/5 criteria met. - Follow-up: waiting for explicit human merge approval.