Closed (won't fix)
Project:
Drupal.org security advisory coverage applications
Component:
module
Priority:
Major
Category:
Task
Assigned:
Unassigned
Reporter:
Created:
7 Sep 2014 at 23:58 UTC
Updated:
23 Nov 2014 at 05:24 UTC
Jump to comment: Most recent
Comments
Comment #1
PA robot commentedThere are some errors reported by automated review tools, did you already check them? See http://pareview.sh/pareview/httpgitdrupalorgsandboxchunglk2333945git
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 #2
chunglk commentedI clean code of GDrive File module.
Comment #3
pgautam commented1) First of all you should create a branch out of your module currently you are working on master branch.
2) Test run your module on pareview.sh to check coding standard and best practices.
3) Create an admin setting form where in you can make admin to enter url and email which you passed static in your module, on line 82 and 147 respectively in gdrive_file.module.
Comment #4
chunglk commentedI'm done #3
Comment #5
PA robot commentedClosing due to lack of activity. If you are still working on this application, you should fix all known problems and then set the status to "Needs review". (See also the project application workflow).
I'm a robot and this is an automated message from Project Applications Scraper.