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
Use invisible recaptcha #224
Conversation
@vidya-ram there is a confusion between |
You need |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
-
If you must change the method signature, issue PRs in every repo that calls the method.
-
We're maintaining feature parity with the Bootstrap templates since they are not discontinued until all the Bootstrap apps have been shut down or migrated.
@@ -101,13 +101,26 @@ | |||
{%- endif %} | |||
{%- endmacro %} | |||
|
|||
{% macro rendersubmit(buttons=[], css_class="", tabindex='', cancel_url='', style='horiz', csrf_error=false) -%} | |||
{% macro rendersubmit(form, ref_id, buttons=[], css_class="", tabindex='', cancel_url='', style='horiz', csrf_error=false) -%} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Changing the parameters will break a lot of code. Add this to the end as optional values. https://github.com/search?q=org%3Ahasgeek+rendersubmit&type=Code
document.getElementById("{{ ref_id }}").submit(); | ||
} | ||
} | ||
</script> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This function can be added to widgetscripts
macro and form, ref_id need not be included as parameters in rendersubmit
Added the scripts to |
If this is required everywhere, why is this config? It should be just code. |
`update` will override app config with default values.
Fixes #143.
This essentially forces invisible recaptcha wherever RecaptchaField is used. On any client app that uses this, it's required to set this in the config -