From bca71604fbe4e75d01f0fb0be788273ef7c58cd1 Mon Sep 17 00:00:00 2001
From: GotPPay <bilal@saburly.com>
Date: Thu, 25 Jan 2018 17:42:17 +0100
Subject: [PATCH] Allow only letters for question name

---
 web/src/App.js                      | 16 +++++++++-------
 web/src/components/IntentDetails.js |  5 ++++-
 2 files changed, 13 insertions(+), 8 deletions(-)

diff --git a/web/src/App.js b/web/src/App.js
index a91c5f6..b2ef726 100644
--- a/web/src/App.js
+++ b/web/src/App.js
@@ -132,7 +132,6 @@ class App extends Component {
     this.setState ({
       selectedIntent: selectedIntent,
       selectedIndex: index,
-      launchRequest: false,
     });
   }
 
@@ -216,14 +215,18 @@ class App extends Component {
   }
 
   handleSaveIntentClick (selectedIntent) {
+    console.log("Save intent");
     let newAllIntentsJSON = JSON.stringify (this.state.allIntents);
     let newAllIntents = JSON.parse (newAllIntentsJSON);
+    
+    let resolveState = null;
+    let rejectState = this.state;
+    rejectState.waiting = false;
 
-    let newState = null;
     if (this.state.selectedIndex === NEW_INTENT_SELECTED_INDEX) {
       //new intent
       newAllIntents.push (selectedIntent);
-      newState = {
+      resolveState = {
         allIntents: newAllIntents,
         selectedIntent: selectedIntent,
         selectedIndex: newAllIntents.length - 1,
@@ -231,18 +234,17 @@ class App extends Component {
       };
     } else {
       newAllIntents[this.state.selectedIndex] = selectedIntent;
-      newState = {
+      resolveState = {
         allIntents: newAllIntents,
         selectedIntent: selectedIntent,
-        waiting: false,
       };
     }
     this.setState ({waiting: true});
     this.sendSkill (
       newAllIntents,
       true,
-      newState,
-      {waiting: false},
+      resolveState,
+      rejectState,
       this.state.invocationName,
       this.state.invocationAnswer,
       this.state.contactEmail,
diff --git a/web/src/components/IntentDetails.js b/web/src/components/IntentDetails.js
index a395bf7..0552775 100644
--- a/web/src/components/IntentDetails.js
+++ b/web/src/components/IntentDetails.js
@@ -126,7 +126,10 @@ class IntentDetails extends Component {
   handleIntentNameEdit(e){
     if (e.length === INTENT_NAME_MAX_LENGTH) return;
     let newIntent = this.state.intent;
-    newIntent.intentName = e.replace(/\s/g, ''); //remove all spaces from intent name
+    //Allow question name with only letters, and with one character minimum
+    if (/^[a-z]+$/i.test(e)){
+      newIntent.intentName = e;
+    }
     this.setState({intent: newIntent});
   }
 }