From 2034966d6dd09656f8f8c552fc61b0158abbd76d Mon Sep 17 00:00:00 2001 From: saqib mir Date: Mon, 7 Sep 2026 16:35:47 +0530 Subject: [PATCH] docs(012-admin-list-views): note the knowledge-governance follow-up Co-Authored-By: Claude Sonnet 5 --- specs/012-admin-list-views/checklists/requirements.md | 8 ++++++++ 1 file changed, 8 insertions(+) diff --git a/specs/012-admin-list-views/checklists/requirements.md b/specs/012-admin-list-views/checklists/requirements.md index aeabcf9..68f3ce1 100644 --- a/specs/012-admin-list-views/checklists/requirements.md +++ b/specs/012-admin-list-views/checklists/requirements.md @@ -50,3 +50,11 @@ only ever been written as free strings across the pause/resume/breach-detection code paths. Centralized it in `orchestration/sla/mapper/sla-run-status.ts` since this feature is the first caller that needs to validate against it, not just write it. +- **Follow-up (post-implementation)**: while building supporthub-web's own knowledge-governance + screen against this feature's own spirit, found a fourth same-shaped gap this spec's own scope + didn't originally name: `GET /knowledge/retrieve` (004-product-knowledge) only ever returns + `status: 'published'` entries — a governance screen that needs to see and publish a *draft* + entry had no endpoint to list it at all. Added `GET /admin/products/:externalProductId/ + knowledge` directly to the knowledge module (not this feature's own routes, since it lives + where `KnowledgeEntry` itself does) in a small follow-up commit, same spirit as this spec's + three original endpoints.