Closed (won't fix)
Project:
Drupal.org security advisory coverage applications
Component:
module
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Reporter:
Created:
28 Apr 2015 at 09:53 UTC
Updated:
21 Jul 2015 at 10:25 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/httpgitdrupalorgsandboxabdesignuk2426765git
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
ziomizar commentedHi aburrows,
Thanks for your module.
- You have to set the version of drupal in the title like [D7] Datedropper.
- Add the link to git command in the issue description
- You dont have a default branch in your repository
- README.txt dont follow the indentation of the README.txt template
- Your plugin dir contain a LICENCE that is not GPL see this document
- In general 3rd party libraries are forbidden, so do not commit any. Instead, document for your users how to find and install the library themselves.
- in line 25 of datedropper.theme.inc seem that there is an error:
('id', 'name', 'value', 'size', 'maxlength', 'placeholder'));should be like this?
array('id', 'name', 'value', 'size', 'maxlength', 'placeholder'));- There is a lot of errors in pareview http://pareview.sh/pareview/httpgitdrupalorgsandboxabdesignuk2426765git
Comment #3
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.