MMT-3899: Temporarily removing Permissions Pages (#1299)

* MMT-3899: Temporarily removing Permissions Pages * MMT-3899: Tests and lint
nasa · Sep 17, 2024 · 81eff52 · 81eff52
1 parent ab76ebf
commit 81eff52
Show file tree

Hide file tree

Showing 3 changed files with 35 additions and 19 deletions.
diff --git a/static/src/js/App.jsx b/static/src/js/App.jsx
@@ -16,8 +16,8 @@ import MetadataFormPage from '@/js/pages/MetadataFormPage/MetadataFormPage'
 import OrderOptionFormPage from '@/js/pages/OrderOptionFormPage/OrderOptionFormPage'
 import OrderOptionListPage from '@/js/pages/OrderOptionListPage/OrderOptionListPage'
 import OrderOptionPage from '@/js/pages/OrderOptionPage/OrderOptionPage'
-import PermissionListPage from '@/js/pages/PermissionListPage/PermissionListPage'
-import PermissionPage from '@/js/pages/PermissionPage/PermissionPage'
+// Import PermissionListPage from '@/js/pages/PermissionListPage/PermissionListPage'
+// Import PermissionPage from '@/js/pages/PermissionPage/PermissionPage'
 import ProviderPermissionsPage from '@/js/pages/ProviderPermissionsPage/ProviderPermissionsPage'
 import ProvidersPage from '@/js/pages/ProvidersPage/ProvidersPage'
 import RevisionListPage from '@/js/pages/RevisionListPage/RevisionListPage'
@@ -31,7 +31,7 @@ import ErrorPageNotFound from '@/js/components/ErrorPageNotFound/ErrorPageNotFou
 import Layout from '@/js/components/Layout/Layout'
 import LayoutUnauthenticated from '@/js/components/LayoutUnauthenticated/LayoutUnauthenticated'
 import Notifications from '@/js/components/Notifications/Notifications'
-import PermissionFormPage from '@/js/pages/PermissionFormPage/PermissionFormPage'
+// Import PermissionFormPage from '@/js/pages/PermissionFormPage/PermissionFormPage'
 import PublishPreview from '@/js/components/PublishPreview/PublishPreview'
 import TemplateForm from '@/js/components/TemplateForm/TemplateForm'
 import TemplateList from '@/js/components/TemplateList/TemplateList'
@@ -140,20 +140,36 @@ export const App = () => {
             },
             {
               path: '/permissions',
-              element: <PermissionListPage />
+              element: <Navigate replace to="/404" />
             },
+            // {
+            //   path: '/permissions',
+            //   element: <PermissionListPage />
+            // },
             {
               path: '/permissions/new',
-              element: <PermissionFormPage />
+              element: <Navigate replace to="/404" />
             },
+            // {
+            //   path: '/permissions/new',
+            //   element: <PermissionFormPage />
+            // },
             {
               path: '/permissions/:conceptId/edit',
-              element: <PermissionFormPage />
+              element: <Navigate replace to="/404" />
             },
+            // {
+            //   path: '/permissions/:conceptId/edit',
+            //   element: <PermissionFormPage />
+            // },
             {
               path: '/permissions/:conceptId',
-              element: <PermissionPage />
+              element: <Navigate replace to="/404" />
             },
+            // {
+            //   path: '/permissions/:conceptId',
+            //   element: <PermissionPage />
+            // },
             {
               path: '/order-options',
               element: <OrderOptionListPage />

diff --git a/static/src/js/components/Layout/Layout.jsx b/static/src/js/components/Layout/Layout.jsx
@@ -138,11 +138,11 @@ const Layout = ({ className, displayNav }) => {
                                   {
                                     to: '/templates/collections',
                                     title: 'Templates'
-                                  },
-                                  {
-                                    to: '/permissions',
-                                    title: 'Permissions'
                                   }
+                                  // {
+                                  //   to: '/permissions',
+                                  //   title: 'Permissions'
+                                  // }
                                 ]
                               },
                               {

diff --git a/static/src/js/components/Layout/__tests__/Layout.test.jsx b/static/src/js/components/Layout/__tests__/Layout.test.jsx
@@ -103,11 +103,11 @@ describe('Layout component', () => {
               {
                 to: '/templates/collections',
                 title: 'Templates'
-              },
-              {
-                title: 'Permissions',
-                to: '/permissions'
               }
+              // {
+              //   title: 'Permissions',
+              //   to: '/permissions'
+              // }
             ]
           },
           {
@@ -215,11 +215,11 @@ describe('Layout component', () => {
                 {
                   to: '/templates/collections',
                   title: 'Templates'
-                },
-                {
-                  title: 'Permissions',
-                  to: '/permissions'
                 }
+                // {
+                //   title: 'Permissions',
+                //   to: '/permissions'
+                // }
               ]
             },
             {