Turnstile's api.js is not getting loaded when form is loaded through Drupal dialog

I am getting the same issue when loading the normal webform page with the setting "Use Ajax" turned on in the webform settings.

Thank you, @gobind-singh - as soon as reCAPTCHA 3.2 is released (or whatever implements the fixes in that issue), we'll port them to this module.

stefanos.petrakis → made their first commit to this issue’s fork.

stefanos.petrakis → changed the visibility of the branch 3330710-PoC-render-when-empty to hidden.

stefanos.petrakis → changed the visibility of the branch 3330710-PoC-render-when-empty to active.

I got a small PoC working that may solve the issue(s) reported here.

For the time, Merge request !3 is available for testing if somebody would have the curiosity/time.

I will move this to Needs Review to get some eyes moving, the maintainers can decide to move this back to Postponed if they feel that this issue should follow the outcomes of 🐛 Ajax support / Use behaviors Fixed

Thank you, Stefanos! I've merged it, and added it to a dev release for testing. Once it's RTBC, I'll make it a supported release.

This patch doesn't work when AJAX is not enabled, unfortunately, including other non-webform forms.

All right, let's work on this more then. Is there more to know apart from disabling AJAX?

Unfortunately, I'm not sure--my own use cases don't use AJAX, so my testing is a bit limited.

At least some steps to reproduce?
I tried with a simple webform without AJAX, got a console warning, but, the submission worked.

Alternatively, could you try to replace the following in turnstile.libraries.yml:

    - core/jquery

to

    - core/drupal.ajax

If that solves the problem you are having (related to the console warnings), then I have an idea how to update the PR.

I have similar issue and this MR fixes it for me (I'm using it exclusively for webform with AJAX enabled), thank you

greatmatter: Could you provide more information where it's not working (exact example) or steps to reproduce?

@parisek I was just trying to troubleshoot the other issue (we don't utilize it this way). I'll probably have to refactor the module, but since so much of it was taken from the reCAPTCHA module, I'll hold off until it supports ^11.

I believe the root of the problem is that the Turnstile library is being attached directly to the HTML head, which doesn't exist for AJAX responses. Instead, I've converted the Turnstile library to a Drupal library and added it as a dependency of the element. This solves the problem for my dynamically-loaded form.

Thank you for your work on this--will test & review!

Fixed the problem for us.

That said the latest release does not apply any more.

Apologies; the latest release should have all of these patches.

Great, confirmed!

Automatically closed - issue fixed for 2 weeks with no activity.

Of the two merge requests, only one was merged. An issue still remains when the entire form is loaded via ajax as the #attached => html_head does not get handled via ajax so the necessary javascript is never loaded.

See this merge request: https://git.drupalcode.org/project/turnstile/-/merge_requests/5/diffs?co...

The attached patch solves the issue and is a slightly modified from the above so that it applies cleanly to the current stable branch.

Here is the correct patch. The above was from the original merge request.