Skip to content

Commit

Permalink
this is the part of #21 where we'll use the message instead of the di…
Browse files Browse the repository at this point in the history
…rect newsletter shortcode
  • Loading branch information
jonathanstegall committed Nov 8, 2019
1 parent 6b72daf commit f4b023d
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 3 deletions.
2 changes: 1 addition & 1 deletion front-page.php
Original file line number Diff line number Diff line change
Expand Up @@ -64,7 +64,7 @@

<aside class="o-content-message o-content-message-homepage">
<article class="o-content-message-body o-content-message-homepage-body">
<?php echo do_shortcode( '[newsletter_form placement="frontpage" content_before="<header><h3 class=\'a-signup-title-homepage\'>Stay informed</h3><h4>Sign up for our daily newsletter</h4></header>" in_content_label="Enter your email address or see our <a href=\'/newsletters/\'>other free newsletter options</a>" in_content_label_placement="after"]' ); ?>
<?php do_action( 'wp_message_inserter', 'homepage_middle' ); ?>
</article>
</aside>

Expand Down
3 changes: 1 addition & 2 deletions template-parts/content.php
Original file line number Diff line number Diff line change
Expand Up @@ -74,8 +74,7 @@
<?php minnpost_share_buttons( 'bottom' ); ?>

<?php if ( 'on' !== get_post_meta( get_the_ID(), '_mp_remove_newsletter_signup_from_display', true ) ) : ?>
<?php /* TODO: we would change this to instead run the 'message' plugin, which would have content around the shortcode, and would contain the shortcode itself */ ?>
<?php echo do_shortcode( '[newsletter_form placement="instory" content_before="<h3 class=\'a-signup-title\'>Get MinnPost in your email inbox:</h3>" content_after="<p>You can also <a href=\'/newsletters/\'>learn about all our free newsletter options</a>.</p>"]' ); ?>
<?php do_action( 'wp_message_inserter', 'article_bottom' ); ?>
<?php endif; ?>

<?php
Expand Down

0 comments on commit f4b023d

Please sign in to comment.