-
Notifications
You must be signed in to change notification settings - Fork 42
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
Update Old ETWP Upsell Notices #3131
Update Old ETWP Upsell Notices #3131
Conversation
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.
Look's good. Should we change the class_exists
code to did_action
for ETP?
@@ -78,23 +80,10 @@ public function show_on_attendees_page() { | |||
return; | |||
} | |||
|
|||
$has_tickets_plus = class_exists( '\Tribe__Tickets_Plus__Main', false ); | |||
$has_wallet_plus = class_exists( '\TEC\Tickets_Wallet_Plus\Plugin', false ); | |||
$has_tickets_plus = did_action( 'tec_container_registered_provider_Tribe__Tickets_Plus__Service_Provider' ); |
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.
We should ticket adding a "tec-tickets-plus-loaded" hook if we need to go this route.
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.
ft_integration
tests are failing...
π« Ticket
ET-2170
ποΈ Description
After merging ETWP into ETP, we found two upsell notices for ETWP in ET. This PR modifies them to be ETP notices.
π₯ Artifacts
βοΈ Checklist
npm run changelog
to add changelog file(s).wpunit
orintegration
tests.wpunit
orintegration
tests.