reCAPTCHA
reCAPTCHA copied to clipboard
PaulMiami
Add support for reCAPTCHA Invisible
Would you be accepting this as a pull request? I might have some time to add it.
I should be able to create a pull request by the end of next week.
Outlook voor Android downloaden
On Mon, Dec 4, 2017 at 11:41 PM +0100, "dotnetshadow" [email protected] wrote:
@SanderDeclerck any luck with this?
— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub, or mute the thread.
I did some tests. The validator does not need to change. The form data is the same as for the standard recaptcha. Only a new tag helper is needed.
Invisible recaptcha is bound to the submit button of a form. My proposal would be to create two taghelpers.
First one: <input for-recaptcha-form>:
<form id="my-form">
<!-- some input fields -->
<input for-recaptcha-form="my-form" type="submit" value="Submit form" />
</form>
Second one: <recaptcha-invisible-script form-id />:
@section Scripts {
<recaptcha-invisible-script form-id="my-form" />
}
I could have it implemented by the end of next week.
@SanderDeclerck thank you so much for this
Do you have a myget of your version?
@dotnetshadow no, I don't have it on myget, and I don't plan to either.
Just wait a bit, give @PaulMiami some time to test and pull the changes.
If you want to use it earlier, then you could clone my fork and add a project dependency - or you could build the project and reference the dll manually.
I don't think there is much sense in creating a myget package just for a single build. One NuGet package for this project is enough: PaulMiami.AspNetCore.Mvc.Recaptcha. It makes it clear which version is the latest.
@SanderDeclerck yes you are right, I was just wondering from a testing point of view but yes I will wait thanks again for your efforts. Much appreciated
You can always clone my fork if you wish to test. There's also a sample project in the solution.
@DJSADERS I have merge @SanderDeclerck #24 into a new branch, you can get it from there. I have identified some issues that I am working on, I don't want to give an eta as I don't really have time to work on it but I'll update this issue when I'll update the new branch...
