Commit fc45a504 authored by Jordan Frankfurt's avatar Jordan Frankfurt Committed by GitHub

fix: log the error message instead of [object Object] (#6773)

* fix: log the error message instead of [object Object]

add tina's rec

pr review

zzmp input

* pr feedback
parent 052cc694
...@@ -8,7 +8,7 @@ import { useContract } from 'hooks/useContract' ...@@ -8,7 +8,7 @@ import { useContract } from 'hooks/useContract'
import { useSingleCallResult } from 'lib/hooks/multicall' import { useSingleCallResult } from 'lib/hooks/multicall'
import ms from 'ms.macro' import ms from 'ms.macro'
import { useCallback, useEffect, useMemo, useState } from 'react' import { useCallback, useEffect, useMemo, useState } from 'react'
import { UserRejectedRequestError } from 'utils/errors' import { toReadableError, UserRejectedRequestError } from 'utils/errors'
import { didUserReject } from 'utils/swapErrorToUserReadableMessage' import { didUserReject } from 'utils/swapErrorToUserReadableMessage'
const PERMIT_EXPIRATION = ms`30d` const PERMIT_EXPIRATION = ms`30d`
...@@ -86,7 +86,7 @@ export function useUpdatePermitAllowance( ...@@ -86,7 +86,7 @@ export function useUpdatePermitAllowance(
if (didUserReject(e)) { if (didUserReject(e)) {
throw new UserRejectedRequestError(`${symbol} permit allowance failed: User rejected signature`) throw new UserRejectedRequestError(`${symbol} permit allowance failed: User rejected signature`)
} }
throw new Error(`${symbol} permit allowance failed: ${e instanceof Error ? e.message : e}`) throw toReadableError(`${symbol} permit allowance failed:`, e)
} }
}, [account, chainId, nonce, onPermitSignature, provider, spender, token]) }, [account, chainId, nonce, onPermitSignature, provider, spender, token])
} }
...@@ -6,3 +6,11 @@ export class UserRejectedRequestError extends Error { ...@@ -6,3 +6,11 @@ export class UserRejectedRequestError extends Error {
this.name = 'UserRejectedRequestError' this.name = 'UserRejectedRequestError'
} }
} }
export function toReadableError(errorText: string, error: unknown) {
if (typeof error === 'object' && error !== null) {
const e = error as Error & { reason?: string }
return new Error(`${errorText} 👺 ${e.message ?? e.reason ?? 'unknown'}`)
}
return new Error(`${errorText} 👺 ${error}`)
}
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment