Bugfix/Add check for setError (#2721)

add check for setError
This commit is contained in:
Henry Heng 2024-06-25 17:54:38 +01:00 committed by GitHub
parent 1367f095d4
commit 96dfedde6e
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
7 changed files with 17 additions and 18 deletions

View File

@ -155,7 +155,7 @@ export default function FlowListMenu({ chatflow, isAgentCanvas, setError, update
await updateChatflowApi.request(chatflow.id, updateBody) await updateChatflowApi.request(chatflow.id, updateBody)
await updateFlowsApi.request() await updateFlowsApi.request()
} catch (error) { } catch (error) {
setError(error) if (setError) setError(error)
enqueueSnackbar({ enqueueSnackbar({
message: typeof error.response.data === 'object' ? error.response.data.message : error.response.data, message: typeof error.response.data === 'object' ? error.response.data.message : error.response.data,
options: { options: {
@ -194,7 +194,7 @@ export default function FlowListMenu({ chatflow, isAgentCanvas, setError, update
await updateChatflowApi.request(chatflow.id, updateBody) await updateChatflowApi.request(chatflow.id, updateBody)
await updateFlowsApi.request() await updateFlowsApi.request()
} catch (error) { } catch (error) {
setError(error) if (setError) setError(error)
enqueueSnackbar({ enqueueSnackbar({
message: typeof error.response.data === 'object' ? error.response.data.message : error.response.data, message: typeof error.response.data === 'object' ? error.response.data.message : error.response.data,
options: { options: {
@ -226,7 +226,7 @@ export default function FlowListMenu({ chatflow, isAgentCanvas, setError, update
await chatflowsApi.deleteChatflow(chatflow.id) await chatflowsApi.deleteChatflow(chatflow.id)
await updateFlowsApi.request() await updateFlowsApi.request()
} catch (error) { } catch (error) {
setError(error) if (setError) setError(error)
enqueueSnackbar({ enqueueSnackbar({
message: typeof error.response.data === 'object' ? error.response.data.message : error.response.data, message: typeof error.response.data === 'object' ? error.response.data.message : error.response.data,
options: { options: {

View File

@ -77,7 +77,7 @@ const APIKeyDialog = ({ show, dialogProps, onCancel, onConfirm, setError }) => {
onConfirm() onConfirm()
} }
} catch (error) { } catch (error) {
setError(error) if (setError) setError(error)
enqueueSnackbar({ enqueueSnackbar({
message: `Failed to add new API key: ${ message: `Failed to add new API key: ${
typeof error.response.data === 'object' ? error.response.data.message : error.response.data typeof error.response.data === 'object' ? error.response.data.message : error.response.data
@ -116,7 +116,7 @@ const APIKeyDialog = ({ show, dialogProps, onCancel, onConfirm, setError }) => {
onConfirm() onConfirm()
} }
} catch (error) { } catch (error) {
setError(error) if (setError) setError(error)
enqueueSnackbar({ enqueueSnackbar({
message: `Failed to save API key: ${ message: `Failed to save API key: ${
typeof error.response.data === 'object' ? error.response.data.message : error.response.data typeof error.response.data === 'object' ? error.response.data.message : error.response.data

View File

@ -100,7 +100,7 @@ const AssistantVectorStoreDialog = ({ show, dialogProps, onCancel, onConfirm, on
}, [listAssistantVectorStoreApi.data]) }, [listAssistantVectorStoreApi.data])
useEffect(() => { useEffect(() => {
if (getAssistantVectorStoreApi.error) { if (getAssistantVectorStoreApi.error && setError) {
setError(getAssistantVectorStoreApi.error) setError(getAssistantVectorStoreApi.error)
} }
// eslint-disable-next-line react-hooks/exhaustive-deps // eslint-disable-next-line react-hooks/exhaustive-deps
@ -152,7 +152,7 @@ const AssistantVectorStoreDialog = ({ show, dialogProps, onCancel, onConfirm, on
} }
setLoading(false) setLoading(false)
} catch (error) { } catch (error) {
setError(error) if (setError) setError(error)
enqueueSnackbar({ enqueueSnackbar({
message: `Failed to delete Vector Store: ${ message: `Failed to delete Vector Store: ${
typeof error.response.data === 'object' ? error.response.data.message : error.response.data typeof error.response.data === 'object' ? error.response.data.message : error.response.data
@ -198,7 +198,7 @@ const AssistantVectorStoreDialog = ({ show, dialogProps, onCancel, onConfirm, on
} }
setLoading(false) setLoading(false)
} catch (error) { } catch (error) {
setError(error) if (setError) setError(error)
enqueueSnackbar({ enqueueSnackbar({
message: `Failed to add new Vector Store: ${ message: `Failed to add new Vector Store: ${
typeof error.response.data === 'object' ? error.response.data.message : error.response.data typeof error.response.data === 'object' ? error.response.data.message : error.response.data
@ -249,8 +249,7 @@ const AssistantVectorStoreDialog = ({ show, dialogProps, onCancel, onConfirm, on
} }
setLoading(false) setLoading(false)
} catch (error) { } catch (error) {
console.error('error=', error) if (setError) setError(error)
setError(error)
enqueueSnackbar({ enqueueSnackbar({
message: `Failed to save Vector Store: ${ message: `Failed to save Vector Store: ${
typeof error.response.data === 'object' ? error.response.data.message : error.response.data typeof error.response.data === 'object' ? error.response.data.message : error.response.data

View File

@ -40,7 +40,7 @@ const LoadAssistantDialog = ({ show, dialogProps, onCancel, onAssistantSelected,
}, [getAllAvailableAssistantsApi.data]) }, [getAllAvailableAssistantsApi.data])
useEffect(() => { useEffect(() => {
if (getAllAvailableAssistantsApi.error) { if (getAllAvailableAssistantsApi.error && setError) {
setError(getAllAvailableAssistantsApi.error) setError(getAllAvailableAssistantsApi.error)
} }
// eslint-disable-next-line react-hooks/exhaustive-deps // eslint-disable-next-line react-hooks/exhaustive-deps

View File

@ -71,14 +71,14 @@ const AddEditCredentialDialog = ({ show, dialogProps, onCancel, onConfirm, setEr
}, [getSpecificComponentCredentialApi.data]) }, [getSpecificComponentCredentialApi.data])
useEffect(() => { useEffect(() => {
if (getSpecificCredentialApi.error) { if (getSpecificCredentialApi.error && setError) {
setError(getSpecificCredentialApi.error) setError(getSpecificCredentialApi.error)
} }
// eslint-disable-next-line react-hooks/exhaustive-deps // eslint-disable-next-line react-hooks/exhaustive-deps
}, [getSpecificCredentialApi.error]) }, [getSpecificCredentialApi.error])
useEffect(() => { useEffect(() => {
if (getSpecificComponentCredentialApi.error) { if (getSpecificComponentCredentialApi.error && setError) {
setError(getSpecificComponentCredentialApi.error) setError(getSpecificComponentCredentialApi.error)
} }
// eslint-disable-next-line react-hooks/exhaustive-deps // eslint-disable-next-line react-hooks/exhaustive-deps
@ -132,7 +132,7 @@ const AddEditCredentialDialog = ({ show, dialogProps, onCancel, onConfirm, setEr
onConfirm(createResp.data.id) onConfirm(createResp.data.id)
} }
} catch (error) { } catch (error) {
setError(error) if (setError) setError(error)
enqueueSnackbar({ enqueueSnackbar({
message: `Failed to add new Credential: ${ message: `Failed to add new Credential: ${
typeof error.response.data === 'object' ? error.response.data.message : error.response.data typeof error.response.data === 'object' ? error.response.data.message : error.response.data
@ -184,7 +184,7 @@ const AddEditCredentialDialog = ({ show, dialogProps, onCancel, onConfirm, setEr
onConfirm(saveResp.data.id) onConfirm(saveResp.data.id)
} }
} catch (error) { } catch (error) {
setError(error) if (setError) setError(error)
enqueueSnackbar({ enqueueSnackbar({
message: `Failed to save Credential: ${ message: `Failed to save Credential: ${
typeof error.response.data === 'object' ? error.response.data.message : error.response.data typeof error.response.data === 'object' ? error.response.data.message : error.response.data

View File

@ -161,7 +161,7 @@ const ToolDialog = ({ show, dialogProps, onUseTemplate, onCancel, onConfirm, set
}, [getSpecificToolApi.data]) }, [getSpecificToolApi.data])
useEffect(() => { useEffect(() => {
if (getSpecificToolApi.error) { if (getSpecificToolApi.error && setError) {
setError(getSpecificToolApi.error) setError(getSpecificToolApi.error)
} }
// eslint-disable-next-line react-hooks/exhaustive-deps // eslint-disable-next-line react-hooks/exhaustive-deps

View File

@ -111,7 +111,7 @@ const AddEditVariableDialog = ({ show, dialogProps, onCancel, onConfirm, setErro
onConfirm(createResp.data.id) onConfirm(createResp.data.id)
} }
} catch (err) { } catch (err) {
setError(err) if (setError) setError(err)
enqueueSnackbar({ enqueueSnackbar({
message: `Failed to add new Variable: ${ message: `Failed to add new Variable: ${
typeof error.response.data === 'object' ? error.response.data.message : error.response.data typeof error.response.data === 'object' ? error.response.data.message : error.response.data
@ -156,7 +156,7 @@ const AddEditVariableDialog = ({ show, dialogProps, onCancel, onConfirm, setErro
onConfirm(saveResp.data.id) onConfirm(saveResp.data.id)
} }
} catch (error) { } catch (error) {
setError(err) if (setError) setError(err)
enqueueSnackbar({ enqueueSnackbar({
message: `Failed to save Variable: ${ message: `Failed to save Variable: ${
typeof error.response.data === 'object' ? error.response.data.message : error.response.data typeof error.response.data === 'object' ? error.response.data.message : error.response.data