fix(rbac): checking rerenders

Author: imjulianeral

example/ts-fire-example/components/router/Shield.tsx

@@ -43,7 +43,7 @@ export function Shield({ children }: Props) {
         accessRoute: '/dashboard',
       },
     },
-    userRole: value?.role ?? 'USER',
+    userRole: value?.role as string,
   }
 
   if (error) return <p>{error}</p>

example/ts-fire-example/package-lock.json

@@ -11,7 +11,7 @@
   "dependencies": {
     "firebase": "^8.7.0",
     "next": "11.0.1",
-    "next-shield": "^2.0.7",
+    "next-shield": "^2.0.8",
     "react": "17.0.2",
     "react-dom": "17.0.2",
     "react-firebase-hooks": "^3.0.4",

example/ts-fire-example/package.json

@@ -65,18 +65,19 @@ export function NextShield<
     RBAC && userRole && verifyPath(RBAC[userRole].grantedRoutes, pathname)
   const access = getAccessRoute(RBAC, userRole, accessRoute, loginRoute)
 
+  console.log({
+    userRole,
+    access,
+    isAuth,
+    isLoading,
+    loginRoute,
+    pathIsPrivate,
+    pathIsPublic,
+    pathIsHybrid,
+    pathIsAuthorized,
+  })
+
   useEffect(() => {
-    console.log({
-      userRole,
-      access,
-      isAuth,
-      isLoading,
-      loginRoute,
-      pathIsPrivate,
-      pathIsPublic,
-      pathIsHybrid,
-      pathIsAuthorized,
-    })
     if (!isAuth && !isLoading && pathIsPrivate) replace(loginRoute)
     if (isAuth && !isLoading && pathIsPublic) replace(access)
     if (isAuth && userRole && !isLoading && !pathIsHybrid && !pathIsAuthorized)
@@ -99,8 +100,10 @@ export function NextShield<
     ((isLoading || isAuth) && pathIsPublic) ||
     ((isLoading || userRole) && !pathIsAuthorized) ||
     (isLoading && pathIsHybrid)
-  )
+  ) {
+    console.log('hey')
     return <>{LoadingComponent}</>
+  }
 
   return <>{children}</>
 }