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

…-template Hotfix/email settings on template
alleyinteractive · Nov 8, 2023 · 4631bc8 · 4631bc8
2 parents 7e553fd + 485d51f
commit 4631bc8
Show file tree

Hide file tree

Showing 4 changed files with 3 additions and 7 deletions.
diff --git a/blocks/email-settings/edit.tsx b/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,

diff --git a/components/emailTypeSelector/index.tsx b/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]);

diff --git a/plugin.php b/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

diff --git a/src/assets.php b/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;
 	}