From 87afc3595c4fdb4da72ba3f3a38e6af7715e0957 Mon Sep 17 00:00:00 2001 From: Tobbe Lundberg Date: Mon, 18 Jul 2022 04:27:10 +0200 Subject: [PATCH] Minor updates after writing PR description --- packages/auth/src/authFactory.ts | 1 + packages/auth/src/authImplementations/netlify.ts | 4 ++++ 2 files changed, 5 insertions(+) diff --git a/packages/auth/src/authFactory.ts b/packages/auth/src/authFactory.ts index 953c35b90e2c..7c4d76b68000 100644 --- a/packages/auth/src/authFactory.ts +++ b/packages/auth/src/authFactory.ts @@ -38,5 +38,6 @@ export function createAuthentication< const useAuth = createUseAuth(AuthContext) const AuthProvider = createAuthProvider(AuthContext, authImplementation) + // TODO: Do we really need to return AuthContext here? return { AuthContext, AuthProvider, useAuth } } diff --git a/packages/auth/src/authImplementations/netlify.ts b/packages/auth/src/authImplementations/netlify.ts index 77af4f3fe67e..86b8fa9821bc 100644 --- a/packages/auth/src/authImplementations/netlify.ts +++ b/packages/auth/src/authImplementations/netlify.ts @@ -1,5 +1,7 @@ import type * as NetlifyIdentityNS from 'netlify-identity-widget' +import { isBrowser } from '@redwoodjs/prerender/browserUtils' + import { createAuthentication } from 'src/authFactory' // TODO: // In the future, when this is a separate package, we can import the full thing @@ -15,6 +17,8 @@ type User = NetlifyIdentityNS.User export function createNetlifyAuth(netlifyIdentity: NetlifyIdentity) { const authImplementation = createNetlifyAuthImplementation(netlifyIdentity) + isBrowser && netlifyIdentity.init() + return createAuthentication< User, User | null,