Add error reporting for newsletter template export fields

assets/js/src/newsletter_editor/components/save.js

@@ -1,12 +1,13 @@
 define([
     'newsletter_editor/App',
     'mailpoet',
+    'notice',
     'backbone',
     'backbone.marionette',
     'jquery',
     'blob',
     'filesaver'
-  ], function(App, MailPoet, Backbone, Marionette, jQuery, Blob, FileSaver) {
+  ], function(App, MailPoet, Notice, Backbone, Marionette, jQuery, Blob, FileSaver) {
 
   "use strict";
 
@@ -144,12 +145,18 @@ define([
       var templateName = this.$('.mailpoet_export_template_name').val(),
           templateDescription = this.$('.mailpoet_export_template_description').val();
 
-      console.log('Exporting template with ', templateName, templateDescription);
+      if (templateName === '') {
-      Module.exportTemplate({
+        MailPoet.Notice.error(App.getConfig().get('translations.templateNameMissing'));
-        name: templateName,
+      } else if (templateDescription === '') {
-        description: templateDescription,
+        MailPoet.Notice.error(App.getConfig().get('translations.templateDescriptionMissing'));
-      });
+      } else {
-      this.hideExportTemplate();
+        console.log('Exporting template with ', templateName, templateDescription);
+        Module.exportTemplate({
+          name: templateName,
+          description: templateDescription,
+        });
+        this.hideExportTemplate();
+      }
     },
     hideOptionContents: function() {
       this.hideSaveAsTemplate();

views/newsletter/form.html

@@ -1239,6 +1239,10 @@
           '<%= __('Test email successfully sent!') %>',
         unknownErrorOccurred:
           '<%= __('An unknown error occurred, please check your settings.') %>',
+        templateNameMissing:
+          '<%= __('Please add a template name') %>',
+        templateDescriptionMissing:
+          '<%= __('Please add a template description') %>',
       },
       sidepanelWidth: '331px',
       validation: {