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

Missing a mandatory field when creating a new Data Provider should not alter the type #1406

Open
MontrealSergiy opened this issue Jul 8, 2024 · 0 comments · May be fixed by #1407
Open

Missing a mandatory field when creating a new Data Provider should not alter the type #1406

MontrealSergiy opened this issue Jul 8, 2024 · 0 comments · May be fixed by #1407
Assignees

Comments

@MontrealSergiy
Copy link
Contributor

MontrealSergiy commented Jul 8, 2024

To reproduce try to create a Data Provider, say DataLad type, yet miss a mandatory field (such as name or DataLad url) to trigger form validation errors. Using already existing provider name would work too.

Expected: the type persist on the form
Observed: type changes (on the production to Loris provider, on my machine Carmin provider)

Note: few less important field, such as "Files can be browsed only by members of this project (browsable DPs only)" field seems to lost, but this should rather go to another issues

This form is admin/manager only, yet
an admin user would assume that when he entered data, it persist, or at he least does not get altered.

@MontrealSergiy MontrealSergiy changed the title Missing fields when creating a new Data Provider might change type Missing a mandatory field when creating a new Data Provider should not alter the type Jul 8, 2024
@MontrealSergiy MontrealSergiy self-assigned this Jul 8, 2024
MontrealSergiy added a commit to MontrealSergiy/cbrain that referenced this issue Jul 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
1 participant