feat: feature flagging moved over to redesign#1140
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: ASSERTIVE Plan: Pro Plus Run ID: 📒 Files selected for processing (3)
📝 WalkthroughSummary by CodeRabbit
WalkthroughThis PR adds a new system-admin-only feature control interface that allows toggling top-level features (Knowledge Center and Program Hub) and dependent sub-features. The change introduces a new admin page accessible via ChangesSystem Admin Feature Control Page
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches📝 Generate docstrings
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
CK-7vn
left a comment
There was a problem hiding this comment.
Couple things, not big at all, nice work man.
carddev81
left a comment
There was a problem hiding this comment.
Amazing work @corypride !! Thanks for getting this done
Ports the Feature Control (feature flagging) admin page from the original frontend to the redesign. System admins can now toggle platform-wide features on and off directly from the redesigned UI.
Changes include:
New
FeatureControl.tsxpage under/feature-control, accessible only toSystemAdminroleToggle support for top-level features (Knowledge Center, Program Hub & Tracking) and Knowledge Center sub-features (Request Content Button, Helpful Links, Videos)
Confirmation dialog before applying any toggle, with a statewide impact warning when disabling resident-facing features
Sub-features are visually disabled (opacity + Switch disabled) when their parent feature is off
"Feature Control" nav link added to
Sidebar.tsxadmin nav, visible only to system admins, usingAdjustmentsHorizontalIconsystemAdminRoutesroute group added toapp-routes.tsxgating the route toUserRole.SystemAdminRelated issues: https://app.asana.com/1/1201607307149189/project/1209460078641109/task/1214641676425184
Screenshot(s)
Additional context
The API calls (
PUT auth/features/:keyandPUT auth/page-features/:key) were already in place from the original frontend — this commit only brings the UI into the redesign. No backend changes were made.RELATED INFO
https://app.asana.com/1/1201607307149189/project/1209460078641109/task/1214641676425184