@planningcenter/chat-react-native 3.21.2-rc.1 → 3.21.2-rc.3

package/src/hooks/use_suspense_api.ts

@@ -9,7 +9,7 @@ import { ApiCollection, ApiResource, ResourceObject } from '../types'
 import { GetRequest, RequestData } from '../utils/client'
 import { App, useApiClient } from './use_api_client'
 import { Log } from '../utils'
-import { ApiError } from '../types/api_primitives'
+import { FailedResponse } from '../types/api_primitives'
 import { ResponseError } from '../utils/response_error'
 
 interface SuspenseGetOptions extends GetRequest {
@@ -18,7 +18,7 @@ interface SuspenseGetOptions extends GetRequest {
 }
 
 export type SuspenseGetQueryOptions<T extends ResourceObject | ResourceObject[]> = Omit<
-  UseSuspenseQueryOptions<ApiResource<T>, ApiError>,
+  UseSuspenseQueryOptions<ApiResource<T>, FailedResponse>,
   'queryKey' | 'queryFn'
 >
 
@@ -29,7 +29,7 @@ export const useSuspenseGet = <T extends ResourceObject | ResourceObject[]>(
   type Resource = ApiResource<T>
   const apiClient = useApiClient()
 
-  const { data, ...query } = useSuspenseQuery<Resource, ApiError>({
+  const { data, ...query } = useSuspenseQuery<Resource, FailedResponse>({
     queryKey: getRequestQueryKey(args),
     queryFn: ({ queryKey }) => {
       const [url, d, headers, app = 'chat'] = queryKey as RequestQueryKey
@@ -104,7 +104,7 @@ export const useSuspensePaginator = <T extends ResourceObject>(
 
 const throwResponseError = (error: unknown) => {
   if (error instanceof Response) {
-    throw new ResponseError(error as ApiError)
+    throw new ResponseError(error as FailedResponse)
   }
 
   return Promise.reject(error)

package/src/types/api_primitives.ts

@@ -23,7 +23,7 @@ export type ApiCollection<Type = ResourceObject> = {
   meta: CollectionMeta
 }
 
-export interface ApiError extends Response {
+export interface FailedResponse extends Response {
   errors: ErrorObject[]
 }
 

package/src/utils/client/client.ts

@@ -1,4 +1,4 @@
-import { ApiCollection, ApiError, ApiResource } from '../../types'
+import { ApiCollection, ApiResource, FailedResponse } from '../../types'
 import {
   concatRecords,
   ensureNoQueryParamsInDev,
@@ -10,11 +10,7 @@ import {
 
 import { DeleteRequest, GetRequest, PatchRequest, PostRequest, WalkRequest } from './types'
 
-export interface ResponseError extends Response {
-  errors: ApiError['errors']
-}
-
-export type OnUnauthorizedResponse = (_response: ResponseError) => void
+export type OnUnauthorizedResponse = (_response: FailedResponse) => void
 
 type ClientArgs = {
   version: string
@@ -112,19 +108,19 @@ export class Client {
 
   handleNotOk = async (response: Response) => {
     if (response.status === 401) {
-      const { errors } = await this.parseErrorResponse(response)
+      const errorData = await this.parseErrorResponse(response)
       this.onUnauthorizedResponse?.({
         ...response,
-        errors: errors || [],
-      })
+        errors: errorData.errors || [],
+      } as FailedResponse)
     }
 
     return Promise.reject(response)
   }
 
-  parseErrorResponse = async (response: Response): Promise<Partial<ApiError>> => {
+  parseErrorResponse = async (response: Response): Promise<Partial<FailedResponse>> => {
     try {
-      return (await response.clone().json()) as ApiError
+      return (await response.clone().json()) as FailedResponse
     } catch {
       return {}
     }

package/src/utils/response_error.ts

@@ -1,12 +1,12 @@
-import { ApiError } from '../types/api_primitives'
+import { FailedResponse } from '../types/api_primitives'
 
 export class ResponseError extends Error {
   status: number
   statusText: string
-  errors: ApiError['errors']
-  response: ApiError
+  errors: FailedResponse['errors']
+  response: FailedResponse
 
-  constructor(response: ApiError) {
+  constructor(response: FailedResponse) {
     super(`ResponseError: ${response?.status} ${response?.statusText}`)
     this.name = 'ResponseError'
     this.status = response?.status
@@ -18,7 +18,7 @@ export class ResponseError extends Error {
 
 export const throwResponseError = (error: unknown) => {
   if (error instanceof Response) {
-    throw new ResponseError(error as ApiError)
+    throw new ResponseError(error as FailedResponse)
   }
 
   return Promise.reject(error)