correodirect [correodirect]

Thanks for your application for a CVS account.
Please, review the CVS application requirements here http://drupal.org/cvs-application/requirements

"You must provide a motivation message, which should be a few paragraphs instead of only a few sentences. It should include details about your contribution, its features, how it compares to already existing solutions, links to screenshots or a demo, etc."

Hi
Just put a reply in this post with them.

A two sentences description is few. The requirements talk of few paragraphs; even in the case few paragraphs are too much, two sentences are still few.

Hi

Here are some points I've found doing a review of the code, they aren't sorted by importance:

• .info file: You shouldn't include version info as drupal package system does that for you, neither the folder should have the version in the name, take a look to the info guidelines http://drupal.org/node/231036 package information would be good as well
• Please include a README.txt file (in English) of what the module does and a English version of INSTALL.txt
• .install file: A header comment block should be included just after the php init tag, guidelines: http://drupal.org/node/546
• .module file: A header comment block should be included just after the php init tag, guidelines: http://drupal.org/node/546
• .module file: All comments should be in English and all the literals have to be originally in English as well, and invoked through a t() call, revise this documentation for more info: http://drupal.org/node/539608
• .module file: Please revise indents and coding standards, see http://drupal.org/coding-standards and run your code through coder module and it will check most of these issues for you http://drupal.org/project/coder
• .module file: You should comment each function using Doxygen http://drupal.org/node/1354
• .module file: I see there is a terms of service agreement inside the module, I'm not really sure that the text belongs there, bear in mind that .module file is loaded *constantly* and you don't want to carry that in all calls, you can take the admin menu to a admin.inc file in hook_menu invocation and take the settings to a separate file. Also, all texts should be translatable and originally in English.