-
Notifications
You must be signed in to change notification settings - Fork 35
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
Marketing Notices for Black Friday. #2225
Changes from 50 commits
834bc5c
e163c74
bb50481
2ea5987
798dbd8
6ffbb00
2955ac0
2790e31
3800071
b8c45b6
08efbc1
2740084
4201ac5
21c009d
11ea932
65a55c2
55d63fb
a4b1b4d
44fe9dd
0cc425c
d76111e
557415b
fdab5a4
b239fee
8235425
5ba54c4
37ab593
3cfd0ec
b676cb2
6a053ab
6c5b0b7
3ca07a3
db9d4ee
6612db7
86a55bd
2d84873
fae268d
85b2add
fdbda1a
216664c
951bf72
5bbc440
13c2d9d
afcde72
189ce37
6cc1963
5c9c8ce
ce54952
95d8431
4fb9615
e6182b4
c498e72
110cc02
7c6cf49
3edd0e2
027f5c0
95d8f9a
65c5f20
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,5 @@ | ||
files=("vendor/composer/autoload_files.php" "vendor/composer/autoload_static.php") | ||
search="symfony/deprecation-contracts/function.php" | ||
for file in "${files[@]}"; do | ||
grep -v "$search" "$file" > temp && mv temp "$file" | ||
done |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,4 @@ | ||
Significance: minor | ||
Type: deprecated | ||
|
||
Removed deprecated classes: `Tribe\Admin\Notice\Marketing`, `Tribe\Admin\Notice\Marketing\Black_Friday`, `Tribe\Admin\Notice\Marketing\End_Of_Year_Sale`, `Tribe\Admin\Notice\Marketing\Stellar_Sale`, `Tribe\Admin\Notice\Conditional_Content\`, `Tribe\Admin\Notice\Marketing\Black_Friday`, `Tribe\Admin\Notice\Marketing\End_Of_Year_Sale`, `Tribe\Admin\Notice\Marketing\Stellar_Sale` |
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,6 +1,6 @@ | ||
{ | ||
"name": "tribe-common", | ||
"version": "6.1.0", | ||
"version": "6.2.0", | ||
Camwyn marked this conversation as resolved.
Show resolved
Hide resolved
|
||
"repository": "[email protected]:the-events-calendar/tribe-common.git", | ||
"_resourcepath": "src/resources", | ||
"_domainPath": "lang", | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,287 @@ | ||
<?php | ||
bordoni marked this conversation as resolved.
Show resolved
Hide resolved
|
||
|
||
namespace TEC\Common\Admin\Conditional_Content; | ||
|
||
use TEC\Common\Admin\Entities\Container; | ||
bordoni marked this conversation as resolved.
Show resolved
Hide resolved
|
||
use TEC\Common\Admin\Entities\Div; | ||
use TEC\Common\Admin\Entities\Button; | ||
use TEC\Common\Admin\Settings_Sidebar; | ||
use Tribe\Utils\Element_Attributes as Attributes; | ||
use TEC\Common\Admin\Entities\Link; | ||
use TEC\Common\Admin\Entities\Image; | ||
use TEC\Common\Admin\Settings_Section; | ||
use Tribe\Utils\Date_I18n; | ||
use Tribe\Utils\Element_Classes; | ||
|
||
/** | ||
* Set up for Black Friday promo. | ||
* | ||
* @since TBD | ||
*/ | ||
class Black_Friday extends Datetime_Conditional_Abstract { | ||
use Dismissible_Trait; | ||
|
||
/** | ||
* @inheritdoc | ||
*/ | ||
bordoni marked this conversation as resolved.
Show resolved
Hide resolved
|
||
protected string $slug = 'black-friday-2024'; | ||
|
||
/** | ||
* @inheritdoc | ||
*/ | ||
bordoni marked this conversation as resolved.
Show resolved
Hide resolved
|
||
protected string $start_date = 'November 26th'; | ||
|
||
/** | ||
* @inheritdoc | ||
*/ | ||
bordoni marked this conversation as resolved.
Show resolved
Hide resolved
|
||
protected string $end_date = 'December 3rd'; | ||
|
||
/** | ||
* @inheritdoc | ||
*/ | ||
public function hook(): void { | ||
add_action( 'tec_settings_sidebar_start', [ $this, 'include_sidebar_section' ] ); | ||
add_action( 'tribe_settings_below_tabs', [ $this, 'include_tickets_settings_section' ] ); | ||
add_action( 'wp_ajax_tec_conditional_content_dismiss', [ $this, 'handle_dismiss' ] ); | ||
} | ||
|
||
/** | ||
* @inheritdoc | ||
*/ | ||
protected function get_start_time(): ?Date_I18n { | ||
$date = parent::get_start_time(); | ||
if ( null === $date ) { | ||
return null; | ||
} | ||
|
||
$date = $date->setTime( 4, 0 ); | ||
|
||
return $date; | ||
} | ||
|
||
/** | ||
* @inheritdoc | ||
*/ | ||
protected function get_end_time(): ?Date_I18n { | ||
$date = parent::get_end_time(); | ||
if ( null === $date ) { | ||
return null; | ||
} | ||
|
||
$date = $date->setTime( 4, 0 ); | ||
|
||
return $date; | ||
} | ||
|
||
/** | ||
* @inheritdoc | ||
*/ | ||
protected function should_display(): bool { | ||
if ( $this->has_user_dismissed() ) { | ||
return false; | ||
} | ||
|
||
if ( tec_should_hide_upsell( $this->slug ) ) { | ||
return false; | ||
} | ||
|
||
return parent::should_display(); | ||
} | ||
|
||
/** | ||
* Gets the content for the Black Friday promo. | ||
* | ||
* @since TBD | ||
* | ||
* @return string | ||
*/ | ||
protected function get_wide_banner_html(): string { | ||
$template_args = [ | ||
'image_src' => tribe_resource_url( 'images/hero-section-wide.jpg', false, null, \Tribe__Main::instance() ), | ||
'link' => 'https://evnt.is/tec-bf-2024', | ||
'nonce' => $this->get_nonce(), | ||
'slug' => $this->slug, | ||
]; | ||
|
||
return $this->get_template()->template( 'black-friday', $template_args, false ); | ||
} | ||
|
||
/** | ||
* Render the wide banner HTML. | ||
* | ||
* @since TBD | ||
* | ||
* @return void | ||
*/ | ||
public function render_wide_banner_html(): void { | ||
if ( ! $this->should_display() ) { | ||
return; | ||
} | ||
|
||
/** | ||
* Fires before the wide banner is rendered. | ||
* This hook is used to add additional content before the narrow banner. | ||
* | ||
* @since TBD | ||
* | ||
* @param string $slug The slug of the conditional content. | ||
* @param Black_Friday $this The Black Friday instance. | ||
*/ | ||
do_action( 'tec_conditional_content_black_friday', 'wide_banner', $this ); | ||
|
||
echo $this->get_wide_banner_html(); // phpcs:ignore WordPress.Security.EscapeOutput.OutputNotEscaped,StellarWP.XSS.EscapeOutput.OutputNotEscaped | ||
} | ||
|
||
/** | ||
* Gets the content for the Black Friday promo. | ||
* | ||
* @since TBD | ||
* | ||
* @return string | ||
*/ | ||
protected function get_narrow_banner_html(): string { | ||
$template_args = [ | ||
'image_src' => tribe_resource_url( 'images/hero-section-narrow.jpg', false, null, \Tribe__Main::instance() ), | ||
'link' => 'https://evnt.is/tec-bf-2024', | ||
'is_narrow' => true, | ||
'nonce' => $this->get_nonce(), | ||
'slug' => $this->slug, | ||
]; | ||
|
||
return $this->get_template()->template( 'black-friday', $template_args, false ); | ||
} | ||
|
||
/** | ||
* Render the narrow banner HTML. | ||
* | ||
* @since TBD | ||
* | ||
* @return void | ||
*/ | ||
public function render_narrow_banner_html(): void { | ||
if ( ! $this->should_display() ) { | ||
return; | ||
} | ||
/** | ||
* Fires before the narrow banner is rendered. | ||
* This hook is used to add additional content before the narrow banner. | ||
* | ||
* @since TBD | ||
* | ||
* @param string $slug The slug of the conditional content. | ||
* @param Black_Friday $this The Black Friday instance. | ||
*/ | ||
do_action( 'tec_conditional_content_black_friday', 'narrow_banner', $this ); | ||
|
||
echo $this->get_narrow_banner_html(); // phpcs:ignore WordPress.Security.EscapeOutput.OutputNotEscaped,StellarWP.XSS.EscapeOutput.OutputNotEscaped | ||
} | ||
|
||
/** | ||
* Include the Black Friday promo in the tickets settings section. | ||
* | ||
* @since TBD | ||
* | ||
* @return void | ||
*/ | ||
public function include_tickets_settings_section(): void { | ||
if ( ! $this->should_display() ) { | ||
return; | ||
} | ||
|
||
$page = tribe_get_request_var( 'page' ); | ||
if ( $page !== 'tec-tickets-settings' ) { | ||
return; | ||
} | ||
|
||
$this->render_narrow_banner_html(); | ||
} | ||
|
||
/** | ||
* Replace the opening markup for the general settings info box. | ||
* | ||
* @since TBD | ||
* | ||
* @param Settings_Sidebar $sidebar Sidebar instance. | ||
* | ||
* @return void | ||
*/ | ||
public function include_sidebar_section( $sidebar ): void { | ||
// Check if the content should currently be displayed. | ||
if ( ! $this->should_display() ) { | ||
return; | ||
} | ||
|
||
$year = date_i18n( 'Y' ); | ||
|
||
/** | ||
* Fires before the settings sidebar is rendered. | ||
* This hook is used to add additional content before the narrow banner. | ||
* | ||
* @since TBD | ||
* | ||
* @param string $slug The slug of the conditional content. | ||
* @param Black_Friday $this The Black Friday instance. | ||
*/ | ||
do_action( 'tec_conditional_content_black_friday', 'settings-sidebar', $this ); | ||
|
||
$translated_title = sprintf( | ||
/* translators: %1$s: Black Friday year */ | ||
esc_attr_x( '%1$s Black Friday Sale for The Events Calendar plugins, add-ons and bundles.', 'Alt text for the Black Friday Ad', 'tribe-common' ), | ||
esc_attr( $year ) | ||
); | ||
|
||
$container = new Container(); | ||
|
||
$button_attr = new Attributes( | ||
[ | ||
'style' => 'position: absolute; top: 0; right: 0; background: transparent; border: 0; color: #fff; padding: 0.5em; cursor: pointer;', | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. [phpcs] reported by reviewdog 🐶 |
||
|
||
// Dismiss button attributes. | ||
'data-tec-conditional-content-dismiss-button' => true, | ||
'data-tec-conditional-content-dismiss-slug' => $this->slug, | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. [phpcs] reported by reviewdog 🐶 There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I've seen this too - apparently it breaks if one key is above some length. I haven't tested to find the "sweet spot" There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. [phpcs] reported by reviewdog 🐶 |
||
'data-tec-conditional-content-dismiss-nonce' => $this->get_nonce(), | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. [phpcs] reported by reviewdog 🐶 |
||
] | ||
); | ||
$button = new Button( null, $button_attr ); | ||
$button->add_child( | ||
new Div( new Element_Classes( [ 'dashicons', 'dashicons-dismiss' ] ) ) | ||
); | ||
|
||
$container->add_child( $button ); | ||
$container->add_child( | ||
new Image( | ||
tribe_resource_url( 'images/hero-section-settings-sidebar.jpg', false, null, \Tribe__Main::instance() ), | ||
new Attributes( | ||
[ | ||
'alt' => $translated_title, | ||
'role' => 'presentation', | ||
] | ||
) | ||
) | ||
); | ||
|
||
$sidebar->prepend_section( | ||
( new Settings_Section() ) | ||
->add_elements( | ||
[ | ||
new Link( | ||
'https://evnt.is/tec-bf-2024', | ||
$container, | ||
null, | ||
new Attributes( | ||
[ | ||
'title' => $translated_title, | ||
'target' => '_blank', | ||
'rel' => 'noopener nofollow', | ||
'style' => 'position: relative; display:block;', | ||
|
||
// Dismiss container attributes. | ||
'data-tec-conditional-content-dismiss-container' => true, | ||
] | ||
) | ||
), | ||
] | ||
) | ||
); | ||
} | ||
} |
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.
Should there be another of these? This only covers the deprecations.