fix: Require logged in user for favorites alerts edit route (#14803)

artsy · Nov 6, 2024 · 51d282c · 51d282c
1 parent 8ec8df6
commit 51d282c
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 6 deletions.
diff --git a/src/Apps/Favorites/favoritesRoutes.tsx b/src/Apps/Favorites/favoritesRoutes.tsx
@@ -87,11 +87,6 @@ export const favoritesRoutes: RouteProps[] = [
         onClientSideRender: () => {
           Alerts.preload()
         },
-        onServerSideRender: ({ req, res }) => {
-          if (!req.user) {
-            res.redirect("/")
-          }
-        },
         query: graphql`
           query favoritesRoutesAlertsAppEditQuery {
             me {

diff --git a/src/Server/middleware/userRequiredMiddleware.ts b/src/Server/middleware/userRequiredMiddleware.ts
@@ -4,7 +4,7 @@ const USER_REQUIRED_ROUTES = [
   "/notifications",
   "/orders(.*)",
   "/my-collection/artworks(.*)",
-  "/favorites/alerts",
+  "/favorites/alerts(.*)",
   "/settings/payments",
   "/settings/purchases(.*)",
   "/settings/shipping",