Train Almond: catch syntax errors inside the promise

So we can report them to the user Fixes #23
stanford-oval · Oct 25, 2018 · 512a903 · 512a903
1 parent d7bb3c9
commit 512a903
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 2 deletions.
diff --git a/browser/new-command.js b/browser/new-command.js
@@ -173,7 +173,9 @@ class ThingTalkTrainer {
         } else {
             let thingtalk = $('#input-command-thingtalk').val();
             if (thingtalk.length > 0) {
-                this._learnThingTalk(thingtalk).then(() => {
+                Promise.resolve().then(() => {
+                    return this._learnThingTalk(thingtalk);
+                }).then(() => {
                     $('#thingtalk-group').removeClass('has-error');
                     $('#thingtalk-error').text('');
                     this._updateConfirmation();

diff --git a/browser/trainer.js b/browser/trainer.js
@@ -170,7 +170,9 @@ class ThingTalkTrainer {
         event.preventDefault();
 
         var tt = $('#thingtalk').val();
-        this._learnThingTalk(tt).then((data) => {
+        Promise.resolve().then(() => {
+            return this._learnThingTalk(tt);
+        }).then((data) => {
             $('#results-container').hide();
             $('#thingtalk-editor').addClass('hidden');
             $('#thingtalk-group').removeClass('has-error');