Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Add Invisible Captcha #743

Merged
merged 3 commits into from
May 2, 2024
Merged

Add Invisible Captcha #743

merged 3 commits into from
May 2, 2024

Conversation

markets
Copy link
Collaborator

@markets markets commented May 1, 2024

Closes #741
Closes #742

Changes:

  • integrate Invisible Captcha
  • send email digest to confirmed users only
  • allow bulk delete in /admin/users

@markets markets added the wip label May 1, 2024
@markets
Copy link
Collaborator Author

markets commented May 1, 2024

@sseerrggii that's ready for testing ✔️

@markets markets added needs testing and removed wip labels May 1, 2024
@sseerrggii
Copy link
Contributor

Tested ✔️

I don't know how to test invisible captcha, let's deploy it on production to see if we can avoid bot accounts

Thanks for the PR @markets and for Gem Invisible Captcha (tot queda a casa 🏠 )

@markets
Copy link
Collaborator Author

markets commented May 2, 2024

Nice thanks @sseerrggii! Yep, that captcha is really invisible 👻. OFN (openfoodfoundation/openfoodnetwork#12073) is also using this solution 👌🏼.

I'll merge and release tonight 🚀

@markets markets merged commit 5a2b32c into develop May 2, 2024
6 checks passed
@markets markets deleted the captcha branch May 2, 2024 21:44
@markets markets mentioned this pull request May 2, 2024
# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Only send weekly digest email to a confirmed email users Add Captcha on # form
2 participants