coderabbitai[bot]
commented
1 week ago [!CAUTION]
Review failed
The pull request is closed.
📝 Walkthrough
📝 Walkthrough
## Walkthrough The pull request introduces several changes to the `wp-parsely` project, focusing on enhancing the dashboard functionality and navigation. A new dependency, `react-router-dom`, is added to the `package.json` file. The `Dashboard_Page` class is updated to improve submenu highlighting and includes additional submenu items. Multiple new React components are created to facilitate routing within the dashboard, including `ParselyDashboard`, `DashboardPage`, `TrafficBoostPage`, and `SettingsPage`, each providing a basic structure for their respective views. ## Changes | File Path | Change Summary | |-----------------------------------------------------------|-----------------------------------------------------------------------------------------------------| | `package.json` | Added new dependencies: `"@types/react-router-dom": "^5.3.3"` and `"react-router-dom": "^6.28.0"`; `"lodash.debounce": "^4.0.8"` unchanged. | | `src/UI/class-dashboard-page.php` | Added method `fix_submenu_highlighting`; updated `add_dashboard_page_to_menu` to include three submenus. | | `src/content-helper/dashboard-page/dashboard-page.tsx` | Introduced `ParselyDashboard` component; integrated React Router; added routing and submenu highlighting logic. | | `src/content-helper/dashboard-page/pages/dashboard/page-component.tsx` | Added new component `DashboardPage` for the dashboard view. | | `src/content-helper/dashboard-page/pages/index.tsx` | Exported new components: `DashboardPage`, `TrafficBoostPage`, `SettingsPage`. | | `src/content-helper/dashboard-page/pages/settings/page-component.tsx` | Added new component `SettingsPage` for settings view. | | `src/content-helper/dashboard-page/pages/traffic-boost/page-component.tsx` | Added new component `TrafficBoostPage` for traffic boost view. | ## Possibly related PRs - **#2841**: This PR updates the `package.json` file, which is relevant as it also involves changes to dependencies, although it focuses on different packages. - **#2940**: This PR adds the same new dependency `"react-router-dom": "^6.28.0"` to the `package.json`, directly relating to the changes made in the main PR. - **#2937**: This PR introduces a `Dashboard_Page` class in `src/UI/class-dashboard-page.php`, which is relevant as it enhances the dashboard functionality, similar to the changes in the main PR. - **#2939**: This PR modifies the `Dashboard_Page` class in `src/UI/class-dashboard-page.php` to include user permissions for the Traffic Boost feature, which is related to the dashboard functionality discussed in the main PR. - **#2940**: This PR implements React Router for navigation within the dashboard, which aligns with the changes made in the main PR regarding the integration of React Router. ## Suggested labels `Feature: Traffic Boost`, `javascript`Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
acicovic
Description
This PR extends the work on #2937, by converting the dashboard in a sort of SPA, using React Router to handle the hash routing, and adds some barebones components and effects to handle the navigation.
Motivation and context
Be able to extend the dashboard in different pages, and still being able to navigate to those pages using the native WordPress sidebar menu.
How has this been tested?
Tested locally on my environment by playing the the different pages.
Screenshots (if appropriate)
https://github.com/user-attachments/assets/56e5cde7-c6fd-4778-8e2d-94c1229ad0fc
Summary by CodeRabbit
Release Notes
New Features
Bug Fixes