fix: restore mobile navigation build after icon key refactor - #4
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Caution Review failedThe pull request is closed. ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughComponent điều hướng mobile thêm ánh xạ ChangesĐiều hướng mobile
Estimated code review effort: 2 (Đơn giản) | ~5 phút Poem
✨ Finishing Touches📝 Generate docstrings
🧪 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. Comment |
Summary
"dashboard"was treated as a JSX intrinsic elementRoot cause
Commit
76caa8echangedAppNavigationItem.iconfrom a Lucide component to the string union key"dashboard"and updated the desktop navigation link mapping, butmobile-app-navigation.tsxstill rendereditem.icondirectly as<Icon />.Scope
Only
src/components/layout/mobile-app-navigation.tsxchanges.Validation
No backend, auth, route, environment, or deployment configuration changes.
Summary by CodeRabbit