move audit log to a system page

Author: david-crespo

app/layouts/SiloLayout.tsx

@@ -12,7 +12,6 @@ import {
   Access16Icon,
   Folder16Icon,
   Images16Icon,
-  Logs16Icon,
   Metrics16Icon,
 } from '@oxide/design-system/icons/react'
 
@@ -38,7 +37,6 @@ export default function SiloLayout() {
           { value: 'Images', path: pb.siloImages() },
           { value: 'Utilization', path: pb.siloUtilization() },
           { value: 'Silo Access', path: pb.siloAccess() },
-          { value: 'Audit Logs', path: pb.siloAuditLogs() },
         ]
           // filter out the entry for the path we're currently on
           .filter((i) => i.path !== pathname)
@@ -72,9 +70,6 @@ export default function SiloLayout() {
           <NavLinkItem to={pb.siloAccess()}>
             <Access16Icon /> Silo Access
           </NavLinkItem>
-          <NavLinkItem to={pb.siloAuditLogs()}>
-            <Logs16Icon /> Audit Logs
-          </NavLinkItem>
         </Sidebar.Nav>
       </Sidebar>
       <ContentPane />

app/layouts/SystemLayout.tsx

@@ -12,6 +12,7 @@ import { apiQueryClient } from '@oxide/api'
 import {
   Cloud16Icon,
   IpGlobal16Icon,
+  Logs16Icon,
   Metrics16Icon,
   Servers16Icon,
 } from '@oxide/design-system/icons/react'
@@ -63,6 +64,7 @@ export default function SystemLayout() {
       { value: 'Utilization', path: pb.systemUtilization() },
       { value: 'Inventory', path: pb.sledInventory() },
       { value: 'IP Pools', path: pb.ipPools() },
+      { value: 'Audit Log', path: pb.auditLog() },
     ]
       // filter out the entry for the path we're currently on
       .filter((i) => i.path !== pathname)
@@ -106,6 +108,9 @@ export default function SystemLayout() {
           <NavLinkItem to={pb.ipPools()}>
             <IpGlobal16Icon /> IP Pools
           </NavLinkItem>
+          <NavLinkItem to={pb.auditLog()}>
+            <Logs16Icon /> Audit Log
+          </NavLinkItem>
         </Sidebar.Nav>
       </Sidebar>
       <ContentPane />

app/pages/SiloAuditLogsPage.tsx

@@ -107,7 +107,7 @@ const HighlightJSON = memo(({ jsonString }: { jsonString: string }) => {
   }
 })
 
-export const handle = { crumb: 'Audit Logs' }
+export const handle = { crumb: 'Audit Log' }
 
 export default function SiloAuditLogsPage() {
   const [expandedItem, setExpandedItem] = useState<string | null>(null)
@@ -298,11 +298,11 @@ export default function SiloAuditLogsPage() {
   return (
     <>
       <PageHeader>
-        <PageTitle icon={<Logs24Icon />}>Audit Logs</PageTitle>
+        <PageTitle icon={<Logs24Icon />}>Audit Log</PageTitle>
         <DocsPopover
-          heading="audit logs"
+          heading="audit log"
           icon={<Logs16Icon />}
-          summary="Audit logs provide a record of all system activities, including user actions, API calls, and system events."
+          summary="The audit log provides a record of system activities, including user actions, API calls, and system events."
           links={[docLinks.auditLogs]}
         />
       </PageHeader>

app/routes.tsx

@@ -209,6 +209,10 @@ export const routes = createRoutesFromElements(
             />
           </Route>
         </Route>
+        <Route
+          path="audit-log"
+          lazy={() => import('./pages/SiloAuditLogsPage').then(convert)}
+        />
       </Route>
 
       <Route index loader={() => redirect(pb.projects())} element={null} />
@@ -252,10 +256,6 @@ export const routes = createRoutesFromElements(
         </Route>
 
         <Route path="access" lazy={() => import('./pages/SiloAccessPage').then(convert)} />
-        <Route
-          path="audit-logs"
-          lazy={() => import('./pages/SiloAuditLogsPage').then(convert)}
-        />
       </Route>
 
       {/* PROJECT */}

app/util/path-builder.ts

@@ -107,7 +107,6 @@ export const pb = {
   siloAccess: () => '/access',
   siloImages: () => '/images',
   siloImageEdit: (params: PP.SiloImage) => `${pb.siloImages()}/${params.image}/edit`,
-  siloAuditLogs: () => '/audit-logs',
 
   systemUtilization: () => '/system/utilization',
 
@@ -129,6 +128,8 @@ export const pb = {
   samlIdp: (params: PP.IdentityProvider) =>
     `${pb.silo(params)}/idps/saml/${params.provider}`,
 
+  auditLog: () => '/system/audit-log',
+
   profile: () => '/settings/profile',
   sshKeys: () => '/settings/ssh-keys',
   sshKeysNew: () => '/settings/ssh-keys-new',