coderabbitai[bot]
commented
2 weeks ago 📝 Walkthrough
## Walkthrough The changes in this pull request introduce a new feature called `traffic_boost` across multiple components of the application. This includes updates to the `Dashboard_Page` and `Settings_Page` classes to manage user permissions and settings for the feature. The `Parsely` class is modified to integrate the `traffic_boost` option into its configuration, while the `Permissions` class is enhanced to check user access for this feature. Additionally, JavaScript and test files are updated to support the new functionality and ensure proper validation and permissions testing. ## Changes | File | Change Summary | |------------------------------------|-------------------------------------------------------------------------------------------------------------| | `src/UI/class-dashboard-page.php` | Added private property `$parsely` and constructor to accept `Parsely` instance; modified `add_dashboard_page_placeholder` for `traffic_boost` permission check. | | `src/UI/class-settings-page.php` | Added `traffic_boost` to `Parsely_Settings_Options_Content_Helper`; registered new settings field in `initialize_content_helper_section`. | | `src/class-parsely.php` | Updated `Parsely_Options_Content_Helper` to include `traffic_boost` option; modified `option_defaults` to initialize this option. | | `src/class-permissions.php` | Enhanced `get_pch_permissions_json` and `build_pch_permissions_settings_array` methods to include `TrafficBoost` feature. | | `src/js/admin-settings.ts` | Added checkbox for `traffic_boost` in feature management; updated event handlers to include this checkbox. | | `tests/Integration/PermissionsTest.php` | Added `traffic_boost` to `$features_to_test` array for permission validation. | | `tests/Integration/UI/SettingsPageTest.php` | Updated `transform_pch_options_for_validation` to include `traffic_boost` in validation process. | | `wp-parsely.php` | Updated instantiation of `Dashboard_Page` to accept `$GLOBALS['parsely']` as a parameter. | ## Possibly related PRs - **#2554**: Adds a new configurable feature called `traffic_boost` to the settings page, related to user permissions functionality. - **#2604**: Implements permissions for AI features, including `traffic_boost`, aligning with the permission checks introduced. - **#2671**: Adds integration tests for the `Excerpt Suggestions` feature, relevant to user permissions changes. - **#2672**: Enhances the testing framework for Content Helper options, including permission settings related to this PR. - **#2895**: Modifies permissions logic for Super Admins, relevant to user permissions enhancements. ## Suggested labels `Changelog: Added` ## Suggested reviewers - vaurdanThank 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 adds the Traffic Boost permissions infrastructure.
Permissions can be changed in the plugin's settings page, and (to facilitate manual testing) the Traffic Boost enabled status is currently shown in our dashboard page.
Motivation and context
Being able to set permissions for Traffic Boost, like for other PCH features.
How has this been tested?
Screenshots
Summary by CodeRabbit