chore: Fix some types

seamapi · Jul 24, 2024 · 43e4221 · 43e4221
1 parent cb29667
commit 43e4221
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 5 deletions.
diff --git a/src/lib/seam/components/AccessCodeDetails/AccessCodeDetails.tsx b/src/lib/seam/components/AccessCodeDetails/AccessCodeDetails.tsx
@@ -27,7 +27,7 @@ import { useIsDateInPast } from 'lib/ui/use-is-date-in-past.js'
 export interface AccessCodeDetailsProps extends CommonProps {
   accessCodeId: string
   onEdit: () => void
-  onDelete: () => void
+  onDelete?: () => void
   isBeingRemoved?: boolean
 }
 

diff --git a/src/lib/seam/components/AccessCodeTable/AccessCodeMenu.tsx b/src/lib/seam/components/AccessCodeTable/AccessCodeMenu.tsx
@@ -15,8 +15,6 @@ export interface AccessCodeMenuProps {
   onViewDetails: () => void
   disableEditAccessCode: boolean
   disableDeleteAccessCode: boolean
-  deleteConfirmationVisible: boolean
-  toggleDeleteConfirmation: () => void
 }
 
 export function AccessCodeMenu(props: AccessCodeMenuProps): JSX.Element {
@@ -44,6 +42,11 @@ export function AccessCodeMenu(props: AccessCodeMenuProps): JSX.Element {
   )
 }
 
+interface ContentProps extends AccessCodeMenuProps {
+  deleteConfirmationVisible: boolean
+  toggleDeleteConfirmation: () => void
+}
+
 function Content({
   accessCode,
   onViewDetails,
@@ -53,7 +56,7 @@ function Content({
   onDelete,
   deleteConfirmationVisible,
   toggleDeleteConfirmation,
-}: AccessCodeMenuProps): JSX.Element {
+}: ContentProps): JSX.Element {
   const deleteAccessCode = useDeleteAccessCode()
 
   if (deleteConfirmationVisible) {

diff --git a/src/lib/seam/components/AccessCodeTable/AccessCodeTable.tsx b/src/lib/seam/components/AccessCodeTable/AccessCodeTable.tsx
@@ -105,7 +105,7 @@ export function AccessCodeTable({
         ?.filter((accessCode) => accessCodeFilter(accessCode, searchInputValue))
         ?.map((accessCode) =>
           deletedAccessCodeIds.includes(accessCode.access_code_id)
-            ? { ...accessCode, status: 'removing' }
+            ? { ...accessCode, status: 'removing' as const }
             : accessCode
         )
         ?.sort(accessCodeComparator) ?? [],