#117 dogfood: wire the OCM topic into renderAdminTopic #449

Merged
Cordy merged 2 commits from fix-117-topic into main 2026-09-07 23:44:24 +00:00
Owner

The OCM admin page showed "This section is coming in a later update": renderAdminTopic is an explicit per-topic if/else chain, and the new topic never got a branch — slot, tabs and save button existed but were unreachable, and save buttons are wired from an explicit id list, not a prefix scan. Adds the branch (Settings default, ensureSettingsLoaded for the transplanted toggle, save bar wiring), the slot in the hide-all list, and adm-save-ocm to the save list. CHANGELOG v0.6.140. Found by Nikola on the first click.

The OCM admin page showed "This section is coming in a later update": `renderAdminTopic` is an explicit per-topic if/else chain, and the new topic never got a branch — slot, tabs and save button existed but were unreachable, and save buttons are wired from an explicit id list, not a prefix scan. Adds the branch (Settings default, `ensureSettingsLoaded` for the transplanted toggle, save bar wiring), the slot in the hide-all list, and `adm-save-ocm` to the save list. CHANGELOG v0.6.140. Found by Nikola on the first click.
#117: wire the ocm topic into renderAdminTopic
All checks were successful
ci / test-and-build (pull_request) Successful in 47s
e38a85f55b
renderAdminTopic is an explicit per-topic if/else chain and a topic without a branch falls through to the coming-in-a-later-update pane - the OCM slot, tabs and save button all existed but were unreachable, and the save wiring is an explicit id list rather than a prefix scan. Added the branch (settings default tab, ensureSettingsLoaded for the transplanted toggle, save bar wiring, slot in the hide-all list) and adm-save-ocm to the save list. CHANGELOG v0.6.140. Found by Nikola on the first click - the lesson for the wiki: a new admin topic needs FOUR touch points (ADM_TOPICS, ADM_TABS, the slot HTML, and the renderAdminTopic branch + save list).
Cordy merged commit 36efac4b72 into main 2026-09-07 23:44:24 +00:00
Cordy deleted branch fix-117-topic 2026-09-07 23:44:26 +00:00
Sign in to join this conversation.
No reviewers
No labels
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: Cordy/Cairn#449
No description provided.