Closed (won't fix)
Project:
Drupal.org security advisory coverage applications
Component:
module
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Reporter:
Created:
1 Aug 2013 at 06:51 UTC
Updated:
10 Oct 2013 at 16:24 UTC
This module is used to send the welcome mail to user on his first login. This mail is manageable by admin panel. This will create the tab at the bottom of the Configuration -> People -> Account Setting page, Under the E-mails section, From here admin can set the welcome mail to send the user on his first login in Drupal 7.
Comments
Comment #1
paul rowell commentedHi Krishna1111,
Have a read through the docs on applying for full project: https://drupal.org/node/1011698 - There's some detail missing from your post.
I've downloaded the module anyway and it only appears to have the .info file. Can you commit your other file(s) and we can have a look then :)
Thanks
Comment #2
krishna1111 commentedHi Paul,
Thanks for your quick response, I have committed the complete module, Please take a review.
Thanks,
Krishna1111
Comment #3
krishna1111 commentedHi,
I have added the module to send the user a welcome email on his first login, and that mail is manageable by admin .
This module is used to send the welcome mail to user on his first login. This mail is manageable by admin panel. This will create the tab at the bottom of the Configuration -> People -> Account Setting page, Under the E-mails section, From here admin can set the welcome mail to send the user on his first login in Drupal 7.
Please take a look, need your review.
Thanks,
Krishna1111
Comment #4
PA robot commentedLink to the project page and git clone command are missing in the issue summary, please add them.
We are currently quite busy with all the project applications and we prefer projects with a review bonus. Please help reviewing and put yourself on the high priority list, then we will take a look at your project right away :-)
Also, you should get your friends, colleagues or other community members involved to review this application. Let them go through the review checklist and post a comment that sets this issue to "needs work" (they found some problems with the project) or "reviewed & tested by the community" (they found no major flaws).
I'm a robot and this is an automated message from Project Applications Scraper.
Comment #5
paul rowell commentedHi again,
You need to include the below detail in your original post (you can edit it and add it in)
Automatic review raised a few things: http://ventral.org/pareview/httpgitdrupalorgsandboxkrishna11112055319git
Manual Review
You have a folder within the folder - will just need a little tidy up
welcome_email.module - apart from the things picked up in the automatic review (above) it looks good. There's a few grammar issues.
i.e line 32: "Edit the welcome email to send user, In case of first time login, Welcome E-mail. Available variables are: [site:name], [site:url], [user:name], [user:mail], [site:login-url], [site:url-brief], [user:edit-url], [user:one-time-login-url], [user:cancel-url]." would read better as: "Edit the e-mail messages sent to users when they log in for the first time. Available variables are......"
Thanks :)
Comment #6
PA robot commentedClosing due to lack of activity. Feel free to reopen if you are still working on this application (see also the project application workflow).
I'm a robot and this is an automated message from Project Applications Scraper.