Merge pull request #35 from alleyinteractive/hotfix/email-settings-on-template

Hotfix/email settings on template

Author: mogmarsh

blocks/email-settings/edit.tsx

@@ -54,10 +54,6 @@ interface Option {
  */
 export default function Edit() {
   const [meta, setMeta] = usePostMeta();
-  // const meta = {};
-  // const setMeta = (newMeta: any) => {
-  //   console.log(newMeta);
-  // };
   const {
     nb_newsletter_subject: subject,
     nb_newsletter_preview: preview,

components/emailTypeSelector/index.tsx

@@ -78,7 +78,7 @@ function EmailTypeSelector({
   // Set the meta to be the first option if it's not set. This matches what is already
   // happpening in the UI.
   useEffect(() => {
-    if (!fromNameValue) {
+    if (!fromNameValue && fromNames.length > 0) {
       fromNameHandler(fromNames[0]);
     }
   }, [fromNameHandler, fromNameValue]);

plugin.php

@@ -3,7 +3,7 @@
  * Plugin Name: Newsletter Builder
  * Plugin URI: https://github.com/alleyinteractive/wp-newsletter-builder
  * Description: Interface to manage email newsletters
- * Version: 0.3.1
+ * Version: 0.3.2
  * Author: Alley Interactive
  * Author URI: https://github.com/alleyinteractive/wp-newsletter-builder
  * Requires at least: 5.9

src/assets.php

@@ -102,7 +102,7 @@ function action_enqueue_block_editor_assets() {
 	global $post_type;
 
 	// Only enqueue the script to register the scripts if supported.
-	if ( 'nb_newsletter' !== $post_type ) {
+	if ( 'nb_newsletter' !== $post_type && 'nb_template' !== $post_type ) {
 		return;
 	}