Skip to content

Commit

Permalink
Use new error and warning types
Browse files Browse the repository at this point in the history
  • Loading branch information
razor-x committed Jul 11, 2024
1 parent e40ea5d commit f980e51
Show file tree
Hide file tree
Showing 3 changed files with 18 additions and 13 deletions.
2 changes: 1 addition & 1 deletion src/lib/seam/components/DeviceTable/DeviceHealthBar.tsx
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
import type { Device } from '@seamapi/types/connect'
import type { ConnectedAccountError, DeviceError } from 'seamapi'
import type { ConnectedAccountError, DeviceError } from '@seamapi/types/connect'

import { CheckIcon } from 'lib/icons/Check.js'
import { ExclamationCircleOutlineIcon } from 'lib/icons/ExclamationCircleOutline.js'
Expand Down
12 changes: 8 additions & 4 deletions src/lib/seam/components/common-props.tsx
Original file line number Diff line number Diff line change
@@ -1,18 +1,22 @@
import type { ComponentType } from 'react'
import type {
AccessCodeError,
AccessCodeWarning,
ConnectedAccountError,
ConnectedAccountWarning,
DeviceError,
SeamWarning,
} from 'seamapi'
DeviceWarning,
} from '@seamapi/types/connect'
import type { ComponentType } from 'react'

export interface RequiredCommonProps {
className: string | undefined
onBack: (() => void) | undefined
errorFilter: (
error: ConnectedAccountError | DeviceError | AccessCodeError
) => boolean
warningFilter: (warning: SeamWarning) => boolean
warningFilter: (
warning: ConnectedAccountWarning | DeviceWarning | AccessCodeWarning
) => boolean
disableDeleteAccessCode: boolean | undefined
disableCreateAccessCode: boolean | undefined
disableEditAccessCode: boolean | undefined
Expand Down
17 changes: 9 additions & 8 deletions src/lib/seam/filters.ts
Original file line number Diff line number Diff line change
@@ -1,30 +1,31 @@
import type {
AccessCodeError,
AccessCodeWarning,
ConnectedAccountError,
DeviceError,
SeamWarning,
} from 'seamapi'
DeviceWarning,
} from '@seamapi/types/connect'

type SeamCompositeError = ConnectedAccountError | DeviceError | AccessCodeError
type SeamResourceError = ConnectedAccountError | DeviceError | AccessCodeError

export const accessCodeErrorFilter = (error: SeamCompositeError): boolean => {
export const accessCodeErrorFilter = (error: SeamResourceError): boolean => {
return 'is_access_code_error' in error && error.is_access_code_error
}

export const accessCodeWarningFilter = (_: SeamWarning): boolean => {
export const accessCodeWarningFilter = (_: AccessCodeWarning): boolean => {
return true
}

export const deviceErrorFilter = (error: SeamCompositeError): boolean => {
export const deviceErrorFilter = (error: SeamResourceError): boolean => {
return 'is_device_error' in error && error.is_device_error
}

export const deviceWarningFilter = (_: SeamWarning): boolean => {
export const deviceWarningFilter = (_: DeviceWarning): boolean => {
return true
}

export const connectedAccountErrorFilter = (
error: SeamCompositeError
error: SeamResourceError
): boolean => {
return (
'is_connected_account_error' in error && error.is_connected_account_error
Expand Down

0 comments on commit f980e51

Please sign in to comment.