Skip to content

CL-6843: Remove draft-only Notifications section from settings registry - #356

Merged
TheGreatAxios merged 3 commits into
mainfrom
cl-6843-wire-or-remove-notifications-draft-only-ui
Aug 23, 2026
Merged

TheGreatAxios merged 3 commits into
mainfrom
cl-6843-wire-or-remove-notifications-draft-only-ui

Conversation

@TheGreatAxios

Copy link
Copy Markdown
Contributor

Summary

  • Removes the personal Notifications (chat) section from @corbits/settings-ui's settings registry — draft-only toggles with an honesty toast on Save, same treatment as AgentSection.
  • Keeps notifications-section.tsx as a dormant draft with a re-add condition once a per-user preference store exists. Drops the package export so nothing can mount the fake UI.
  • Workbench-scoped mute/mentions prefs in @corbits/chat-ui are unchanged (those already persist).

Test plan

  • bun test test/section-registry.test.tsx in packages/settings-ui
  • Open Personal Settings — Notifications nav entry should be gone; Account remains
  • Workbench settings → Notifications still loads and persists (chat-ui path)

CL-6843

Honest absence over fake toggles — same pattern as AgentSection. No
per-user preference store backs personal notification prefs yet; keep
the dormant component for re-add when one exists. Workbench-scoped
mute/mentions in chat-ui are unchanged.
…e-notifications-draft-only-ui

# Conflicts:
#	packages/settings-ui/README.md
#	packages/settings-ui/src/section-registry.tsx
@TheGreatAxios
TheGreatAxios merged commit fa2c751 into main Aug 23, 2026
5 checks passed
@TheGreatAxios
TheGreatAxios deleted the cl-6843-wire-or-remove-notifications-draft-only-ui branch August 25, 2026 15:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant