Remove 'Uploading <name>...' from text input on error
This commit is contained in:
parent
af7cbd0846
commit
1b7b869fdb
@ -254,6 +254,11 @@ export function MarkdownInput ({ label, topLevel, groupClassName, onChange, onKe
|
|||||||
const s3Keys = [...text.matchAll(AWS_S3_URL_REGEXP)].map(m => Number(m[1]))
|
const s3Keys = [...text.matchAll(AWS_S3_URL_REGEXP)].map(m => Number(m[1]))
|
||||||
updateImageFees({ variables: { s3Keys } })
|
updateImageFees({ variables: { s3Keys } })
|
||||||
}}
|
}}
|
||||||
|
onError={({ name }) => {
|
||||||
|
let text = innerRef.current.value
|
||||||
|
text = text.replace(`![Uploading ${name}…]()`, '')
|
||||||
|
helpers.setValue(text)
|
||||||
|
}}
|
||||||
>
|
>
|
||||||
<AddImageIcon width={18} height={18} />
|
<AddImageIcon width={18} height={18} />
|
||||||
</ImageUpload>
|
</ImageUpload>
|
||||||
|
@ -137,7 +137,7 @@ export default function ZoomableImage ({ src, srcSet, ...props }) {
|
|||||||
return <ImageOriginal src={originalUrl} onClick={handleClick} {...props} />
|
return <ImageOriginal src={originalUrl} onClick={handleClick} {...props} />
|
||||||
}
|
}
|
||||||
|
|
||||||
export function ImageUpload ({ children, className, onSelect, onSuccess }) {
|
export function ImageUpload ({ children, className, onSelect, onSuccess, onError }) {
|
||||||
const ref = useRef()
|
const ref = useRef()
|
||||||
const toaster = useToast()
|
const toaster = useToast()
|
||||||
|
|
||||||
@ -166,6 +166,7 @@ export function ImageUpload ({ children, className, onSelect, onSuccess }) {
|
|||||||
({ data } = await getSignedPOST({ variables }))
|
({ data } = await getSignedPOST({ variables }))
|
||||||
} catch (e) {
|
} catch (e) {
|
||||||
toaster.danger(e.message || e.toString?.())
|
toaster.danger(e.message || e.toString?.())
|
||||||
|
onError?.({ ...variables, name: file.name, file })
|
||||||
return
|
return
|
||||||
}
|
}
|
||||||
|
|
||||||
@ -184,6 +185,7 @@ export function ImageUpload ({ children, className, onSelect, onSuccess }) {
|
|||||||
if (!res.ok) {
|
if (!res.ok) {
|
||||||
// TODO make sure this is actually a helpful error message and does not expose anything to the user we don't want
|
// TODO make sure this is actually a helpful error message and does not expose anything to the user we don't want
|
||||||
toaster.danger(res.statusText)
|
toaster.danger(res.statusText)
|
||||||
|
onError?.({ ...variables, name: file.name, file })
|
||||||
return
|
return
|
||||||
}
|
}
|
||||||
|
|
||||||
|
Loading…
x
Reference in New Issue
Block a user