Repository navigation
feat: add global ui and modal redux slice (#113) - #131
Conversation
|
Navigate logical layers of code changes, visualize relationships, and explore their blast radius. Warning Review limit reachedNext included review available in 55 minutes. View limit detailsLimit details: You’ve used the included review currently available. You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository. Review configuration: ⚙️ Run configurationConfiguration used: Organization UI Review profile: ASSERTIVE Plan: Advanced Run ID: 📒 Files selected for processing (1)
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: ASSERTIVE Plan: Advanced Run ID: 📒 Files selected for processing (2)
Included review availability: This review used your included allowance. Your plan provides up to 1 included review per hour; 0 remain after this review. 📜 Recent review details🔇 Additional comments (2)
📝 WalkthroughWalkthroughThe pull request adds a typed Redux Toolkit slice for sidebar, preview-device, builder-tab, and modal state. It registers the slice reducer in the Redux store. ChangesUI State
Priority: ⬇️ Low Estimated code review effort: 2 (Simple) | ~10 minutes Change: Feature Suggested reviewers: Merge Risk: ⚪ Minimal · up to The UI state addition appears mergeable after normal checks. Security Architecture ReviewSecurity architecture risk: 🔵 Low · up to The new state controls interface visibility, not deletion or integration permissions. No connected privileged action was identified, but how future modal users will enforce those permissions is not established. Retained concerns Security review detailsSecurity Blast Radius
Trust Boundaries and Controls
Hardening Proposals
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches 💡 1⚔️ Resolve merge conflicts 💡
🧪 Generate unit tests (beta)
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. A rabbit hops beside the state, Comment |
# Conflicts: # apps/web/store/store.ts
Hey team! This PR takes care of #113 by setting up our global UI state using Redux Toolkit.
I've added a new uiSlice that handles all the core interface toggles. It tracks whether the sidebar is open, which device we're previewing (desktop, tablet, or mobile), and which builder tab is currently active. I also set up a clean dictionary to manage our modal states (like the AI prompt, delete confirmations, and integrations) so we can easily pop them open or closed from anywhere in the app.
Everything is fully typed and already hooked up to the main store. Let me know what you think or if you need any tweaks!