franzliedke/fluxbb-new-recaptcha

it needs work on reply/newtopic page

TurkDesk opened this issue · 5 comments

Should have a setting to enable for reply/newtopic for forums that has setting no need register to reply/new topic.

Hmm... so you mean the CAPTCHAs should be shown when creating individual posts, too - not just on registration?

I don't think I will have the time to build that in the coming weeks. But it would be a cool feature.
Would you be interested in trying to send a pull request? :)

Hi,

It's my first time on GitHub so I'm not too familar with it yet.

I actually created a fork about this particular feature just a few days ago. But I'm not entirely satisfied with it, because the code is modified to work in a particular case (reCAPTCHA on login.php), while it should be the particular case to be modified to work with the current code.

I'm thinking of refactoring it a little because of that. Would you be interested in a pull request/merge after that?

Any feedback is welcome.

Sure, I am always happy about pull requests. :)

If you're unsure how to proceed, you can also open the pull request even if the feature is not yet completed or you have questions. The discussion can then take place directly in the pull request, while you finish it.

Closing because of inactivity.

This feature has been published as part of version 2.0.0.