Add minitable to Floating IP side modal (#2630)

oxidecomputer · Jan 10, 2025 · 6cd25c9 · 6cd25c9
1 parent a629300
commit 6cd25c9
Show file tree

Hide file tree

Showing 2 changed files with 52 additions and 10 deletions.
diff --git a/app/forms/floating-ip-edit.tsx b/app/forms/floating-ip-edit.tsx
@@ -6,25 +6,41 @@
  * Copyright Oxide Computer Company
  */
 import { useForm } from 'react-hook-form'
-import { useNavigate, type LoaderFunctionArgs } from 'react-router'
+import { Link, useNavigate, type LoaderFunctionArgs } from 'react-router'
 
-import { apiq, queryClient, useApiMutation, usePrefetchedApiQuery } from '@oxide/api'
+import {
+  apiq,
+  getListQFn,
+  queryClient,
+  useApiMutation,
+  usePrefetchedQuery,
+} from '@oxide/api'
 
 import { DescriptionField } from '~/components/form/fields/DescriptionField'
 import { NameField } from '~/components/form/fields/NameField'
 import { SideModalForm } from '~/components/form/SideModalForm'
 import { HL } from '~/components/HL'
 import { getFloatingIpSelector, useFloatingIpSelector } from '~/hooks/use-params'
 import { addToast } from '~/stores/toast'
+import { EmptyCell } from '~/table/cells/EmptyCell'
+import { IpPoolCell } from '~/table/cells/IpPoolCell'
+import { CopyableIp } from '~/ui/lib/CopyableIp'
+import { PropertiesTable } from '~/ui/lib/PropertiesTable'
+import { ALL_ISH } from '~/util/consts'
 import type * as PP from '~/util/path-params'
 import { pb } from 'app/util/path-builder'
 
 const floatingIpView = ({ project, floatingIp }: PP.FloatingIp) =>
   apiq('floatingIpView', { path: { floatingIp }, query: { project } })
+const instanceList = (project: string) =>
+  getListQFn('instanceList', { query: { project, limit: ALL_ISH } })
 
 EditFloatingIpSideModalForm.loader = async ({ params }: LoaderFunctionArgs) => {
   const selector = getFloatingIpSelector(params)
-  await queryClient.prefetchQuery(floatingIpView(selector))
+  await Promise.all([
+    queryClient.fetchQuery(floatingIpView(selector)),
+    queryClient.fetchQuery(instanceList(selector.project).optionsFn()),
+  ])
   return null
 }
 
@@ -35,10 +51,11 @@ export function EditFloatingIpSideModalForm() {
 
   const onDismiss = () => navigate(pb.floatingIps({ project: floatingIpSelector.project }))
 
-  const { data: floatingIp } = usePrefetchedApiQuery('floatingIpView', {
-    path: { floatingIp: floatingIpSelector.floatingIp },
-    query: { project: floatingIpSelector.project },
-  })
+  const { data: floatingIp } = usePrefetchedQuery(floatingIpView(floatingIpSelector))
+  const { data: instances } = usePrefetchedQuery(
+    instanceList(floatingIpSelector.project).optionsFn()
+  )
+  const instanceName = instances.items.find((i) => i.id === floatingIp.instanceId)?.name
 
   const editFloatingIp = useApiMutation('floatingIpUpdate', {
     onSuccess(_floatingIp) {
@@ -49,7 +66,6 @@ export function EditFloatingIpSideModalForm() {
   })
 
   const form = useForm({ defaultValues: floatingIp })
-
   return (
     <SideModalForm
       form={form}
@@ -66,6 +82,32 @@ export function EditFloatingIpSideModalForm() {
       loading={editFloatingIp.isPending}
       submitError={editFloatingIp.error}
     >
+      <PropertiesTable>
+        <PropertiesTable.IdRow id={floatingIp.id} />
+        <PropertiesTable.DateRow label="Created" date={floatingIp.timeCreated} />
+        <PropertiesTable.DateRow label="Updated" date={floatingIp.timeModified} />
+        <PropertiesTable.Row label="IP Address">
+          <CopyableIp ip={floatingIp.ip} isLinked={false} />
+        </PropertiesTable.Row>
+        <PropertiesTable.Row label="IP Pool">
+          <IpPoolCell ipPoolId={floatingIp.ipPoolId} />
+        </PropertiesTable.Row>
+        <PropertiesTable.Row label="Instance">
+          {instanceName ? (
+            <Link
+              to={pb.instanceNetworking({
+                project: floatingIpSelector.project,
+                instance: instanceName,
+              })}
+              className="link-with-underline group text-sans-md"
+            >
+              {instanceName}
+            </Link>
+          ) : (
+            <EmptyCell />
+          )}
+        </PropertiesTable.Row>
+      </PropertiesTable>
       <NameField name="name" control={form.control} />
       <DescriptionField name="description" control={form.control} />
     </SideModalForm>

diff --git a/app/pages/project/floating-ips/FloatingIpsPage.tsx b/app/pages/project/floating-ips/FloatingIpsPage.tsx
@@ -61,8 +61,8 @@ const instanceList = (project: string) =>
 FloatingIpsPage.loader = async ({ params }: LoaderFunctionArgs) => {
   const { project } = getProjectSelector(params)
   await Promise.all([
-    queryClient.prefetchQuery(fipList(project).optionsFn()),
-    queryClient.prefetchQuery(instanceList(project).optionsFn()),
+    queryClient.fetchQuery(fipList(project).optionsFn()),
+    queryClient.fetchQuery(instanceList(project).optionsFn()),
     // fetch IP Pools and preload into RQ cache so fetches by ID in
     // IpPoolCell can be mostly instant yet gracefully fall back to
     // fetching individually if we don't fetch them all here