-
Notifications
You must be signed in to change notification settings - Fork 11
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: validate submission response required fields #241
base: master
Are you sure you want to change the base?
feat: validate submission response required fields #241
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #241 +/- ##
==========================================
+ Coverage 89.19% 89.29% +0.10%
==========================================
Files 142 143 +1
Lines 2175 2205 +30
Branches 468 476 +8
==========================================
+ Hits 1940 1969 +29
Misses 222 222
- Partials 13 14 +1 ☔ View full report in Codecov by Sentry. |
src/components/FileUpload/index.jsx
Outdated
/> | ||
<Form.Group isInValid> | ||
<Dropzone | ||
multiple |
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.
I know that you didn't change this, but to my knowledge the paragon version of dropzone does not support multiple files. I would double check this functionality and remove the attribute if multiple files are not supported.
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.
Correct. I attempted to upload two files at a time but came across this error Only one upload permitted.
Summary
Text and file upload requirement not being applied when submitting an ORA and it's letting learners submit without meeting the response requirement. This PR validates the required fields