feat: move product catalog to new settings page#183
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: ASSERTIVE Plan: Pro Run ID: 📒 Files selected for processing (5)
WalkthroughA new Settings page is introduced with a Product Catalog section, registered as a lazy-loaded nested route under the dashboard path, and linked from the Home menu. ProductCatalogImport rendering is removed from CreateInvoice and CreateInvoicesBatch pages. ChangesSettings Page Introduction and Product Catalog Relocation
Estimated code review effort: 1 (Trivial) | ~5 minutes Sequence Diagram(s)sequenceDiagram
participant User
participant Home
participant App
participant Settings
User->>Home: click Settings menu item
Home->>App: navigate to /dashboard/settings
App->>Settings: lazy load and render Settings
Settings-->>User: display Product Catalog section
Possibly related PRs
Suggested labels: Poem
🚥 Pre-merge checks | ✅ 4✅ Passed checks (4 passed)
✨ Finishing Touches🧪 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 |
Addressed Issues:
Fixes #(TODO:issue number)
Screenshots/Recordings:
freecompress-setting.page.mp4
Additional Notes:
This PR introduces a new Settings page to the dashboard and extracts the
ProductCatalogImportcomponent from the invoice creation pages (CreateInvoice.jsxandCreateInvoicesBatch.jsx).Previously, the product catalog management interface cluttered the invoice forms. By migrating this to a dedicated Settings page, we provide a cleaner and more focused user experience. Users can now manage their product catalog independently, while the autocomplete functionality on the invoice forms continues to pull data seamlessly from the catalog.
Changes Made
Settings.jsxto serve as a centralized place for managing user settings and the product catalog./dashboard/settingswith Reactlazyloading inApp.jsx.Home.jsx.ProductCatalogImportcomponent fromCreateInvoice.jsxandCreateInvoicesBatch.jsxto reduce visual clutter.AI Usage Disclosure:
We encourage contributors to use AI tools responsibly when creating Pull Requests. While AI can be a valuable aid, it is essential to ensure that your contributions meet the task requirements, build successfully, include relevant tests, and pass all linters. Submissions that do not meet these standards may be closed without warning to maintain the quality and integrity of the project. Please take the time to understand the changes you are proposing and their impact. AI slop is strongly discouraged and may lead to banning and blocking. Do not spam our repos with AI slop.
Check one of the checkboxes below:
I have used the following AI models and tools: TODO
Checklist
Summary by CodeRabbit