-
Notifications
You must be signed in to change notification settings - Fork 4
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat(365): Add bulk upload #382
Conversation
76be0ab
to
b26db29
Compare
b26db29
to
83d5ed3
Compare
} | ||
}; | ||
|
||
const [isUploadComplete, setIsUploadComplete] = useState<boolean>(false); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Might need a better state solution soon like contextAPI. This is ok for now.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
sounds good
const defaultErrorText = 'This is not a valid file type.' | ||
|
||
const filePreviews = [] | ||
if (files) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
const filePreviews = files?.length
? files.map((file) => {
const imageId = makeSafeForID(file.name)
const key = `filePreview_${imageId}`
return (
<FilePreview
key={key}
imageId={imageId}
file={file}
/>
)
})
: []
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ill make a follow up for this
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
Description
Screenshots (if applicable)
Related Issues
[Link any related issues or tasks from your project management system.]
Checklist