reagent-project / reagent-forms

Bootstrap form components for Reagent
339 stars 78 forks source link

Fields missing from form cursor under advanced compilation #41

Closed rsslldnphy closed 9 years ago

rsslldnphy commented 9 years ago

I appreciate this is a bit vague, but thought I'd create an issue anyway in case you had an idea what might be causing it.

I'm running into a problem where some fields in a reagent-forms form are not present in the form's state atom on submit. It only happens to some fields, not all of them (but always seems to be the same fields in my experience) and it only happens under advanced compilation. So far I've only seen password fields affected, although not sure that's relevant.

Removing reagent-forms and wiring up the forms manually makes everything work fine.

I'm afraid our code isn't open source so I can't share it with you, but we're using

[reagent "0.5.0-alpha2"]
[reagent-forms  "0.4.3"]
rsslldnphy commented 9 years ago

I've actually just managed to replicate this without reagent-forms, so closing this. Still no idea what is happening but it appears to be something to do with using inputs of type password - the on-change handler does not seem to be firing under advanced compilation.