Forgot your password is a simple module that improves the user experience for the default Request New Password tab on the user login page.

Project page: http://drupal.org/sandbox/kruser/1444262
Git: git clone --branch 7.x-1.x kruser@git.drupal.org:sandbox/kruser/1444262.git forgot_your_password
For Drupal 7

http://ventral.org/pareview/httpgitdrupalorgsandboxkruser1444262git

Comments

svenaas’s picture

This looks helpful, and I haven't found with the code (which is straightforward, uses t() where I expect it to, etc.). I also confirmed that PAReview isn't reporting any issues.

ankitchauhan’s picture

Status: Needs work » Needs review

hi kruser

This project is bit short, At least 120 lines of code or 5 function should be there.
See the link http://groups.drupal.org/node/195848

misc’s picture

Hi kruser, and welcome with your application.

You need to update your project page with more information and the information above according to our guidelines, see http://drupal.org/node/1187664 - the git repo should not be your private one, it should be git clone http://git.drupal.org/sandbox/kruser/1444262.git forgot_your_password. Are there similar modules on drupal.org?

A manual review:

README.txt
You are misisng README.txt in the MASTER branch that tells which branch to use
Your REDAME.txt in 7.x-1.x should be updated with more information about the project, follow the guidelines for in-project documentation.
Code too short
This project is too short to approve you as git vetted user. We are currently discussing how much code we need, but everything with less than 120 lines of code or less than 5 functions cannot be seriously reviewed. However, we can promote this single project manually to a full project, if you are interested.

Have you considered in submitting a patch to core, instead of doing a module?
Personally I think that the text should be configured in the admin interface instead of stored as strings.

misc’s picture

Status: Needs review » Needs work
klausi’s picture

Status: Needs review » Closed (won't fix)

Closing due to lack of activity. Feel free to reopen if you are still working on this application.

klausi’s picture

Issue summary: View changes

wrong branch