- Issue created by @HitchShock
- last update
over 1 year ago 38 pass - @hitchshock opened merge request.
- Status changed to Needs review
over 1 year ago 5:31pm 17 September 2023 - last update
over 1 year ago 38 pass - last update
over 1 year ago 38 pass - last update
over 1 year ago 38 pass - last update
over 1 year ago 38 pass - last update
over 1 year ago 38 pass - Issue was unassigned.
- Status changed to Needs work
about 1 year ago 6:22am 9 October 2023 - 🇺🇦Ukraine HitchShock Ukraine
Found one mistake in my code:
The upgrade path must be a hook_post_update_NAME function instead of the hook_update_N - last update
about 1 year ago 38 pass - last update
about 1 year ago 38 pass - Status changed to Needs review
about 1 year ago 8:41pm 14 October 2023 - last update
about 1 year ago 38 pass - last update
about 1 year ago 38 pass - last update
about 1 year ago 38 pass - 🇺🇸United States joshua.boltz
I needed this patch to avoid the attached ajax error when I filled in the required fields on my form and submit the form, which uses ajax to submit.
I was not getting this ajax error when I did not first fill in the required fields on my form, I guess because when i did that, it loaded up the captcha widget and then it existed from the loading of the widget?
- 🇺🇦Ukraine HitchShock Ukraine
I think yes, I had a similar AJAX issue on the webform, so it was fixed in the context of this patch too.
- 🇺🇸United States joshua.boltz
I've attached a patch that worked in my case where I was having various issues with the recaptcha displaying and working alongside my #ajax-enabled form submit.
- last update
about 1 year ago 38 pass - 🇺🇸United States joshua.boltz
I have an updated patch that includes a minor update from my previous patch. With the previous patch, I was running into an error stemming from:
reCAPTCHA validation failed, error codes: invalid-input-responseWhich seems related to this question:
https://www.drupal.org/project/simple_recaptcha/issues/3338584#comment-1... 🐛 Drupal 8 invalid-input-response in beta 7 update Needs reviewAnd why the form actions stuff was there and what it was doing.
Anyways, removing the formActions stuff fixed the issue for me, so including the updated patch if it helps anyone else.
- 🇺🇦Ukraine HitchShock Ukraine
Hi @joshuaboltz
I haven't tested your solution yet, but your patch missed the update path for a new setting parameter. See `simple_recaptcha.post_update.php` in the previous patch - Status changed to Needs work
about 1 year ago 2:02pm 6 December 2023