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

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.

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.

None yet

4 participants