✨ Add recaptcha challenge on hasura auth

nhost / hasura-auth

Authentication for Hasura.

https://nhost.io

MIT License

376 stars 111 forks source link

✨ Add recaptcha challenge on hasura auth #428

Closed lechinoix closed 3 months ago

lechinoix commented 9 months ago

Checklist

[x] No breaking changes
[ ] Tests pass
[ ] New features have new tests
[ ] Documentation is updated

Breaking changes

Avoid breaking changes and regressions. If you feel it is unavoidable, make it explicit in your PR comment so we can review it and see how to handle it.

Tests

please make sure your changes pass the current tests (Use the make test or the make watch command).
if you are introducing a new feature, please write as much tests as possible.

Documentation

Please make sure the documentation is updated accordingly, in particular:

Workflows. Workflows are Mermaid sequence diagrams
Schema. The schema in a Mermaid ER diagram
Environment variables. Please adjust the .env.example file accordingly
OpenApi specifications. We are using inline JSDoc annotations

changeset-bot[bot] commented 9 months ago

⚠️ No Changeset found

Latest commit: 535803438c2dab3a484d729edca0987c8108e790

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

lechinoix commented 9 months ago

Hello !

I just implemented Google Recaptcha on my fork, I was wondering if you were interested on merging this.

If it is the case, let me know I'll be happy to take some time to write tests / documentation :)

Have a good one,

Nicolas

dbarrosop commented 9 months ago

thanks! I think this is a nice addition but if you could start by writing a short guide on how to use it at least we could test it very quickly and see how it feels.

Thanks again!

stale[bot] commented 3 months ago

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.