Skip to content
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

894 refactor captcha system #923

Merged
merged 1 commit into from
Jul 3, 2024
Merged

Conversation

davidcoutadeur
Copy link

Closes #894

@davidcoutadeur davidcoutadeur added this to the 1.7.0 milestone Jun 25, 2024
@davidcoutadeur davidcoutadeur self-assigned this Jun 25, 2024
@davidcoutadeur
Copy link
Author

Last version seems ok:

  • refactoring of captcha system in one class
  • unit test

Last thing missing documentation:

  • how to configure the internal captcha
  • how to write your own captcha

@davidcoutadeur davidcoutadeur marked this pull request as draft June 25, 2024 16:06
@davidcoutadeur davidcoutadeur force-pushed the 894-refactor-captcha-system branch 3 times, most recently from 2ea6c7c to e283667 Compare July 2, 2024 14:37
@davidcoutadeur davidcoutadeur marked this pull request as ready for review July 2, 2024 16:52
@davidcoutadeur davidcoutadeur force-pushed the 894-refactor-captcha-system branch from 7eee5ef to 4049d34 Compare July 3, 2024 10:02
@davidcoutadeur davidcoutadeur force-pushed the 894-refactor-captcha-system branch from 4049d34 to bcbc71f Compare July 3, 2024 15:24
@davidcoutadeur
Copy link
Author

documentation is written, unit tests seems ok. I am going to merge this.

@davidcoutadeur davidcoutadeur merged commit fc2a753 into master Jul 3, 2024
1 check passed
@coudot coudot deleted the 894-refactor-captcha-system branch October 3, 2024 05:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

refactor the captcha system for integrating external captcha
1 participant