feat: block access to not logged in users (#365)

openfoodfacts · Dec 4, 2022 · d51074d · d51074d
1 parent 5db8726
commit d51074d
Show file tree

Hide file tree

Showing 3 changed files with 87 additions and 8 deletions.
diff --git a/src/App.js b/src/App.js
@@ -19,6 +19,7 @@ import {
   Home,
   Nutrition,
   FlaggedImages,
+  ShouldLoggedinPage,
   PackagingPage,
 } from "./pages";
 import ResponsiveAppBar from "./components/ResponsiveAppBar";
@@ -168,26 +169,91 @@ export default function App() {
             <Routes>
               <Route path="/" element={<Home />} />
               <Route path="/eco-score" element={<EcoScorePage />} />
-              <Route path="/logos" element={<LogoAnnotationPage />} />
-              <Route path="/logos/search" element={<LogoSearchPage />} />
-              <Route path="/logos/:logoId" element={<LogoUpdatePage />} />
+              <Route
+                path="/logos"
+                element={
+                  userState.isLoggedIn ? (
+                    <LogoAnnotationPage />
+                  ) : (
+                    <ShouldLoggedinPage />
+                  )
+                }
+              />
+              <Route
+                path="/logos/search"
+                element={
+                  userState.isLoggedIn ? (
+                    <LogoSearchPage />
+                  ) : (
+                    <ShouldLoggedinPage />
+                  )
+                }
+              />
+              <Route
+                path="/logos/:logoId"
+                element={
+                  userState.isLoggedIn ? (
+                    <LogoUpdatePage />
+                  ) : (
+                    <ShouldLoggedinPage />
+                  )
+                }
+              />
               <Route
                 path="/logos/product-search"
-                element={<ProductLogoAnnotationPage />}
+                element={
+                  userState.isLoggedIn ? (
+                    <ProductLogoAnnotationPage />
+                  ) : (
+                    <ShouldLoggedinPage />
+                  )
+                }
+              />
+              <Route
+                path="/logos/deep-search"
+                element={
+                  userState.isLoggedIn ? (
+                    <LogoDeepSearch />
+                  ) : (
+                    <ShouldLoggedinPage />
+                  )
+                }
               />
-              <Route path="/logos/deep-search" element={<LogoDeepSearch />} />
 
               <Route path="/settings" element={<SettingsPage />} />
               <Route path="/questions" element={<QuestionsPage />} />
               <Route path="/insights" element={<InsightsPage />} />
               <Route path="*" element={<NotFoundPage />} />
-              <Route path="/nutriscore" element={<NutriscoreValidator />} />
-              <Route path="/nutrition" element={<Nutrition />} />
+              <Route
+                path="/nutriscore"
+                element={
+                  userState.isLoggedIn ? (
+                    <NutriscoreValidator />
+                  ) : (
+                    <ShouldLoggedinPage />
+                  )
+                }
+              />
+              <Route
+                path="/nutrition"
+                element={
+                  userState.isLoggedIn ? <Nutrition /> : <ShouldLoggedinPage />
+                }
+              />
               {showFlaggedImage && (
                 <Route path="/flagged-images" element={<FlaggedImages />} />
               )}
 
-              <Route path="/packaging" element={<PackagingPage />} />
+              <Route
+                path="/packaging"
+                element={
+                  userState.isLoggedIn ? (
+                    <PackagingPage />
+                  ) : (
+                    <ShouldLoggedinPage />
+                  )
+                }
+              />
             </Routes>
           </DevModeContext.Provider>
         </LoginContext.Provider>

diff --git a/src/pages/index.js b/src/pages/index.js
@@ -16,4 +16,5 @@ export { default as LogoDeepSearch } from "./logos/LogoDeepSearch";
 export { default as NutriscoreValidator } from "./nutriscoreValidator";
 export { default as FlaggedImages } from "./flaggedImages";
 
+export { default as ShouldLoggedinPage } from "./shouldLoggedinPage";
 export { default as PackagingPage } from "./packaging";
diff --git a/src/pages/shouldLoggedinPage/index.js b/src/pages/shouldLoggedinPage/index.js
@@ -0,0 +1,12 @@
+import Typography from "@mui/material/Typography";
+import Box from "@mui/material/Box";
+
+const ShouldLoggedinPage = () => (
+  <Box sx={{ p: 2 }}>
+    <Typography variant="h5" component="h1" sx={{ mb: 2 }}>
+      Restricted page
+    </Typography>
+    <Typography>Advenced game are restricted to connected user</Typography>
+  </Box>
+);
+export default ShouldLoggedinPage;