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

Don't submit widget's internal input fields with form data #6

Open
bhch opened this issue Sep 20, 2021 · 2 comments
Open

Don't submit widget's internal input fields with form data #6

bhch opened this issue Sep 20, 2021 · 2 comments
Labels
bug Something isn't working

Comments

@bhch
Copy link
Owner

bhch commented Sep 20, 2021

Currently, all the widgets fields are submitted with the form. But the expected behaviour is that only the JSON data is submitted.

One possible solution is using data-name instead of name on inputs.

@bhch bhch added the bug Something isn't working label Sep 20, 2021
@patdahl
Copy link

patdahl commented May 8, 2023

It is good to know that only the JSON data needs to be submitted. When playing around with django-jsonform, I realized that 'multiselect' values are not sent unlike the rest of the inputs. Because of how the widget is implemented, there are no name attributes on any of the input elements.

Apparently this is not even the intended function, but the inconsistency was a bit confusing.

I always feel bad leaving comments on public repos that are even remotely negative -- library is awesome!

@bhch
Copy link
Owner Author

bhch commented May 8, 2023

This has been on my to-do list for so long but I've been putting it off. I'll try to fix it in the next release.

But thanks for commenting. I did not find it negative at all.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants