Skip to content
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

fix(forms): Cannot save record with file=required field #4398

Open
dtroelofsprins opened this issue Oct 25, 2024 · 0 comments · May be fixed by #4402
Open

fix(forms): Cannot save record with file=required field #4398

dtroelofsprins opened this issue Oct 25, 2024 · 0 comments · May be fixed by #4402
Assignees
Labels
app:forms bug Something isn't working

Comments

@dtroelofsprins
Copy link
Contributor

What version of EMX2 are you using (see footer)
v11.23.1

Describe the bug
Saving a record where a field of type=file which is required is included is not possible:
image

To Reproduce
Steps to reproduce the behavior:

  1. Go to 'https://emx2.dev.molgenis.org'
  2. Create a new schema using PET_STORE template
  3. Go to schema, User table and put picture column required=True
  4. Save schema and go to tables => User
  5. Enter a new row and file in data, include a file
  6. Save the record => see the error
  7. When file is not required, record can be saved without any problem.

Expected behavior
A column with type=file should also be included as a required field. OR it should be made clear in the manual that this is not possible.

@dtroelofsprins dtroelofsprins added bug Something isn't working app:forms labels Oct 25, 2024
@connoratrug connoratrug self-assigned this Oct 25, 2024
@connoratrug connoratrug linked a pull request Oct 25, 2024 that will close this issue
3 tasks
@connoratrug connoratrug linked a pull request Oct 25, 2024 that will close this issue
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
app:forms bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants