Skip to content

Conversation

@subdesign
Copy link

I added the following functions:

  • optional email activation
  • user registration with reCaptcha
  • forgot password
  • alert message email to admin if user is locked out (optional)

@danmontgomery
Copy link
Owner

I dig this. Need to find time in the next few days to look it over, and I'll merge it in.

@subdesign
Copy link
Author

All right, feel free to modify or prettify the code if needed.

@CroNiX
Copy link

CroNiX commented May 26, 2012

I'm trying to test this but getting the "To use reCAPTCHA you must get an API key from https://www.google.com/recaptcha/admin/create" message. I do have a key entered, but I'm testing this on localhost and google states any key should work on localhost.

@subdesign
Copy link
Author

I don't clearly understand the problem with reCaptcha, can you explain in a bit more detail?

@CroNiX
Copy link

CroNiX commented May 29, 2012

Call to undefined method Bitauth::get_user_by_email()
Called by Bitauth::generate_forgot_code()

@CroNiX
Copy link

CroNiX commented May 29, 2012

@subdesign I was just testing the base install with your mod, set the recaptcha keys in the config (even though I'm testing on localhost/127.0.0.1 and google says any key will work on localhost), and got the error that I reported above. I've disabled it as I don't need it for what I'm doing, but it didn't work for my tests.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why is this <script> here, I think this should be moved out.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants