collective / collective.easyform

Forms for Plone
https://pypi.org/project/collective.easyform
13 stars 28 forks source link

Migrate ReCaptcha field to not required #407

Closed ThibautBorn closed 10 months ago

ThibautBorn commented 10 months ago

context: ReCaptcha fields may not be required in easyform

mauritsvanrees commented 10 months ago

I don't get it. I would think that ReCaptcha fields should always be required, otherwise they do not work. At least, that is what I expect. Can you explain?

ThibautBorn commented 10 months ago

Yes, that was my assumption as well. But it is stated in the readme here that it should be left unchecked. And when I tested it, the recpatcha would indeed not pass if it was required. While it is enforced even when the check is iggnored.

By default the recaptchas were required in Plone 4, so all forms with recaptchas broke in Easyforms if left unchanged.

mauritsvanrees commented 10 months ago

Released as 4.1.4.