fix: ensure dev settings for url is only visible on dev

netlify · Jun 4, 2019 · 2998060 · 2998060
1 parent c2685bf
commit 2998060
Show file tree

Hide file tree

Showing 2 changed files with 16 additions and 3 deletions.
diff --git a/src/components/app.js b/src/components/app.js
@@ -91,8 +91,13 @@ class App extends Component {
     const page = pages[store.modal.page] || {};
     const pageLinkHandler = () => this.handlePage(page.link);
 
-    if (!store.gotrue) {
-      return <SiteURLForm onSiteURL={this.handleSiteURL} />;
+    if (process.env.NODE_ENV === "development" && store.siteURL === null) {
+      return (
+        <SiteURLForm
+          devMode={store.siteURL != null}
+          onSiteURL={store.siteURL ? this.clearSiteURL : this.handleSiteURL}
+        />
+      )
     }
     if (!store.settings) {
       return;
@@ -127,7 +132,14 @@ class App extends Component {
             {page.link_text}
           </button>
         )}
-        <SiteURLForm devMode="true" onSiteURL={this.clearSiteURL} />
+        {process.env.NODE_ENV === "development" ? (
+          <SiteURLForm
+            devMode={store.siteURL != null}
+            onSiteURL={store.siteURL ? this.clearSiteURL : this.handleSiteURL}
+          />
+        ) : (
+          <div />
+        )}
       </div>
     );
   }

diff --git a/src/netlify-identity.js b/src/netlify-identity.js
@@ -84,6 +84,7 @@ function instantiateGotrue(APIUrl) {
       parts.push("/")
     }
     parts.push(".netlify/identity")
+    store.setSiteURL(siteURL)
     return new GoTrue({ APIUrl: parts.join(""), setCookie: !isLocal })
   }
   if (isLocal) {