Skip to content

Commit c58aa6c

Browse files
committed
Handle error state when settings endpoint fails
1 parent 4e94789 commit c58aa6c

File tree

4 files changed

+7
-4
lines changed

4 files changed

+7
-4
lines changed

example/src/index.js

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,7 @@ import App from './App';
55
import registerServiceWorker from './registerServiceWorker';
66
import netlifyIdentity from 'netlify-identity-widget';
77

8+
window.netlifyIdentity = netlifyIdentity
89
// You must run this once before trying to interact with the widget
910
netlifyIdentity.init()
1011

src/components/app.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -156,7 +156,7 @@ class App extends Component {
156156
showHeader={showHeader}
157157
showSignup={showSignup}
158158
devSettings={!store.gotrue}
159-
loading={store.gotrue && !store.settings}
159+
loading={!store.error && store.gotrue && !store.settings}
160160
isOpen={store.modal.isOpen}
161161
onPage={this.handlePage}
162162
onClose={this.handleClose}

src/netlify-identity.js

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -49,7 +49,8 @@ const netlifyIdentity = {
4949
},
5050
init: options => {
5151
init(options);
52-
}
52+
},
53+
store
5354
};
5455

5556
let queuedIframeStyle = null;

src/state/store.js

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -56,8 +56,9 @@ store.loadSettings = action(function loadSettings() {
5656
.then(action(settings => (store.settings = settings)))
5757
.catch(
5858
action(err => {
59-
console.error("failed to load settings %o", err);
60-
store.error = err;
59+
store.error = new Error(
60+
`Failed to load settings from ${store.gotrue.api.apiURL}`
61+
);
6162
})
6263
);
6364
});

0 commit comments

Comments
 (0)