-
Notifications
You must be signed in to change notification settings - Fork 17
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
Callback option needed #25
base: master
Are you sure you want to change the base?
Conversation
Added addition helper attributes to pass into the rendered element i.e. for specifying a callback
Recaptcha callback
Package name change
Changed repo name back
Repo Name
Fixing CI failures
Any reason you can't specify the callback when loading component or the helper? What you want can already be achieve using |
I don't see how that gets my data-callback attribute onto the view? Or is what you're suggesting a way to get the attribute into the view, and so I would only need to commit the change to recaptcha view element? |
Ok. It works the way you suggested with no alterations to the Helper. Not sure the advantage of having it this way though. |
Remove $xtraAttribs option
Callback option CI fix
Removed short tag
Remove whitespace
Added blank line for phpcs test pass
Hi, what about the other callbacks? it would be useful to have an option for those too. |
Added option for specifying attributes for the Helper i.e. to provide a callback function