-
Notifications
You must be signed in to change notification settings - Fork 74
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
Reject disabled fields #70
base: master
Are you sure you want to change the base?
Conversation
I would love to see this merged. Disabled input fields should not be serialized or submitted to the server. |
Hi @sockmonk can you expand on the usecase as to why disabled input fields should not be serialized? |
I think the idiomatic thing to do is to not submit disabled fields For example http://jsfiddle.net/00bqLtsp/ |
Syphon is only to get you from form to object however. Submission is another beast. I think the middle ground will be somewhere between here and #72 |
true true :) just leaning on the giants for some perspective |
Ah good point @samccone |
Hey @serverglitch would you mind opening this against the major branch? And could you fixup the failing tests please? |
Originally done by @samlown in #18. Separated the disabled field rejection into its own commit.