Skip to content
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

Add support for the ReCaptcha onload callback #15281

Closed
mroskamp opened this issue Feb 8, 2024 · 1 comment · Fixed by #15283
Closed

Add support for the ReCaptcha onload callback #15281

mroskamp opened this issue Feb 8, 2024 · 1 comment · Fixed by #15283
Milestone

Comments

@mroskamp
Copy link
Contributor

mroskamp commented Feb 8, 2024

Is your feature request related to a problem? Please describe.

I use JQuery Unobtrusive Validation to validate my forms. I also use the ReCaptcha feature to protect my forms against robots. I'd like to be able to add validation to the reCAPTCHA form element, which is loaded dynamically and thus can't be added until after the reCAPTCHA is loaded. Fortunately, Google provides an onload parameter that you can provide on the ReCaptcha script (see documentation). Unfortunately, there isn't a convenient way to specify that parameter and when using the ReCaptchaTagHelper.

Describe the solution you'd like

Add an attribute to the ReCaptchaTagHelper for specifying the onload callback.

Describe alternatives you've considered

For my site, I replaced the base Orchard Core ReCaptchaTagHelper with a custom implementation with the onload attribute.

@hishamco
Copy link
Member

hishamco commented Feb 8, 2024

Can you create a PR for it?

mroskamp pushed a commit to mroskamp/OrchardCore that referenced this issue Feb 8, 2024
@sebastienros sebastienros added this to the 1.x milestone Feb 15, 2024
@MikeAlhayek MikeAlhayek modified the milestones: 2.x, 2.0 Sep 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants