Fixed warnings
This commit is contained in:
parent
fbb0dd7b32
commit
caa50709dd
@ -1,4 +1,4 @@
|
||||
import { useState, useContext, useEffect, useRef } from 'react'
|
||||
import { useState, useContext, useEffect, useRef, useCallback } from 'react'
|
||||
import PropTypes from 'prop-types'
|
||||
import { Flex, Alert, Card, Spin, Splitter, Button, Modal, Input } from 'antd'
|
||||
import { LoadingOutlined } from '@ant-design/icons'
|
||||
@ -44,25 +44,28 @@ const TemplateEditor = ({
|
||||
}
|
||||
}
|
||||
|
||||
function reloadPreview(content, testObject = {}, scale = 1) {
|
||||
fetchTemplatePreview(
|
||||
objectData._id,
|
||||
content,
|
||||
testObject,
|
||||
scale,
|
||||
(result) => {
|
||||
setReloadLoading(false)
|
||||
if (result?.error) {
|
||||
setPreviewError(true)
|
||||
setPreviewMessage(result.error)
|
||||
} else {
|
||||
setPreviewError(false)
|
||||
updatePreviewContent(result.html)
|
||||
setPreviewMessage('No issues found.')
|
||||
const reloadPreview = useCallback(
|
||||
(content, testObject = {}, scale = 1) => {
|
||||
fetchTemplatePreview(
|
||||
objectData._id,
|
||||
content,
|
||||
testObject,
|
||||
scale,
|
||||
(result) => {
|
||||
setReloadLoading(false)
|
||||
if (result?.error) {
|
||||
setPreviewError(true)
|
||||
setPreviewMessage(result.error)
|
||||
} else {
|
||||
setPreviewError(false)
|
||||
updatePreviewContent(result.html)
|
||||
setPreviewMessage('No issues found.')
|
||||
}
|
||||
}
|
||||
}
|
||||
)
|
||||
}
|
||||
)
|
||||
},
|
||||
[fetchTemplatePreview, objectData?._id]
|
||||
)
|
||||
|
||||
// Move useEffect to component level and use state to track objectData changes
|
||||
useEffect(() => {
|
||||
@ -70,7 +73,7 @@ const TemplateEditor = ({
|
||||
console.log('PreviewScale', previewScale)
|
||||
reloadPreview(objectData.content, objectData.testObject, previewScale)
|
||||
}
|
||||
}, [objectData, previewScale])
|
||||
}, [objectData, previewScale, reloadPreview])
|
||||
|
||||
return (
|
||||
<>
|
||||
|
||||
Loading…
x
Reference in New Issue
Block a user