New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Remove Sell Course with WooCommerce task and deprecate task class #7572
Conversation
Test the previous changes of this PR with WordPress Playground. |
Test the previous changes of this PR with WordPress Playground. |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## trunk #7572 +/- ##
=========================================
Coverage 51.78% 51.78%
+ Complexity 11323 11321 -2
=========================================
Files 641 641
Lines 48164 48159 -5
Branches 468 468
=========================================
- Hits 24941 24939 -2
+ Misses 22844 22841 -3
Partials 379 379
Continue to review full report in Codecov by Sentry.
|
@@ -9,6 +9,8 @@ | |||
/** | |||
* Sensei_Home_Task_Sell_Course_With_WooCommerce class. | |||
* | |||
* @deprecated $$next-version$$ We're now showing the "Sell your course with Sensei Pro" (Sensei_Home_Task_Pro_Upsell) task instead of this task. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
WDYT of adding a constructor here calling the _deprecated_function
. So if something is still using it will add the warning to the logs.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good suggestion, updated here 9ac6dca
Test the previous changes of this PR with WordPress Playground. |
Test the previous changes of this PR with WordPress Playground. |
* Class constructor. | ||
*/ | ||
public function __construct() { | ||
if ( is_wp_version_compatible( '6.4' ) ) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
TIL!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good and works well!
Resolves #7571
Proposed Changes
Removed usage of Sell your course with WooCommerce task's class and deprecated it.
Testing Instructions
Sell your course with Sensei Pro
instead ofSell your course with WooCommerce
Pre-Merge Checklist