From 782f7eeabca0b4e5b85e04488c12a422ef04464e Mon Sep 17 00:00:00 2001
From: wdransfield <wdransfield@instructure.com>
Date: Fri, 7 Jun 2019 09:04:27 -0600
Subject: [PATCH] Set editing to false on request error

Closes PLAT-4574

Test Plan:
Create an LTI key via pasting JSON. Use invalid JSON on first
attempt. Verify you can then use valid JSON without refreshing.

Change-Id: Ic0be5675951649aeafda3002d47f4aac786354b5
Reviewed-on: https://gerrit.instructure.com/196889
Reviewed-by: Drake Harper <dharper@instructure.com>
Reviewed-by: Marc Phillips <mphillips@instructure.com>
Reviewed-by: Kai Bjorkman <kbjorkman@instructure.com>
QA-Review: Marc Phillips <mphillips@instructure.com>
Tested-by: Jenkins
Product-Review: Weston Dransfield <wdransfield@instructure.com>
---
 app/jsx/developer_keys/actions/ltiKeyActions.js | 1 +
 1 file changed, 1 insertion(+)

diff --git a/app/jsx/developer_keys/actions/ltiKeyActions.js b/app/jsx/developer_keys/actions/ltiKeyActions.js
index 1dd9fa5e0b7..d7ab67f7835 100644
--- a/app/jsx/developer_keys/actions/ltiKeyActions.js
+++ b/app/jsx/developer_keys/actions/ltiKeyActions.js
@@ -124,6 +124,7 @@ actions.saveLtiToolConfiguration = ({
     })
     .catch(error => {
       dispatch(actions.saveLtiToolConfigurationFailed(error))
+      dispatch(developerKeysActions.setEditingDeveloperKey(false))
       $.flashError(error.message)
       return error
     })