From 0354c9fb0d43eff49f9908d0b90d56ea556fc135 Mon Sep 17 00:00:00 2001
From: headphonejames <generalfuzz@gmail.com>
Date: Tue, 21 Feb 2023 19:35:37 -0800
Subject: [PATCH] fix SignInPageWrapper with new sign in hooks

Signed-off-by: headphonejames <generalfuzz@gmail.com>
---
 packages/core-app-api/src/app/AppRouter.tsx | 16 +++++++++++++++-
 1 file changed, 15 insertions(+), 1 deletion(-)

diff --git a/packages/core-app-api/src/app/AppRouter.tsx b/packages/core-app-api/src/app/AppRouter.tsx
index b799983a3b..33de9d479c 100644
--- a/packages/core-app-api/src/app/AppRouter.tsx
+++ b/packages/core-app-api/src/app/AppRouter.tsx
@@ -72,8 +72,22 @@ function SignInPageWrapper({
   const configApi = useApi(configApiRef);
   const basePath = getBasePath(configApi);
 
+  const handleSignInStarted = () => {
+    // no-op
+  };
+
+  const handleSignInFailure = () => {
+    // no-op
+  };
+
   if (!identityApi) {
-    return <Component onSignInSuccess={setIdentityApi} />;
+    return (
+      <Component
+        onSignInSuccess={setIdentityApi}
+        onSignInStarted={handleSignInStarted}
+        onSignInFailure={handleSignInFailure}
+      />
+    );
   }
 
   appIdentityProxy.setTarget(identityApi, {