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

Added functionalities #6

Open
wants to merge 6 commits into
base: master
Choose a base branch
from
Open

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.

@@ -15,6 +15,7 @@
<title>BitAuth: <?php echo $title; ?></title>
</head>
<body>
<script>

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.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants