CVS edit link for ninu_ann

I have been working as a drupal developer for the past 2 and a half years. I have created a number of modules some are similar to the existing with little more customizations. I wanted to donate my modules to drupal after making them standalone.
CommentFileSizeAuthor
#1 ip_tracker.zip2.61 KBOptimusPrime23

Comments

OptimusPrime23’s picture

Title: ninu_ann [ninuann] » New Project name: IP Tracker
Status: Postponed (maintainer needs more info) » Needs review
StatusFileSize
new2.61 KB

This module adds a form to your Drupal site.
This module retrieves the basic information such as country, province and city of the IP address.

dagmar’s picture

Title: New Project name: IP Tracker » ninu_ann [ninuann]
Status: Needs review » Postponed (maintainer needs more info)

Hello, and thanks for applying for a CVS account.

There are several issues with you code that have to be fixed before grant you CVS access:

But first to all, it would be nice to know why you are creating a new module to do the same that this projects are doing:

http://drupal.org/project/ip2cc
http://drupal.org/project/geoip
http://drupal.org/project/geouser
http://drupal.org/project/hostip
http://drupal.org/project/ip2nation
http://drupal.org/project/ip2country
http://drupal.org/project/ip_locator

As the CVS Application docs says: http://drupal.org/cvs-application/requirements

Your new project must not duplicate the work done in existing projects. Instead consider collaborating with existing maintainers. http://drupal.org/node/23789

If you can justify the existence of this new project, I or other users will review the code and give you some advices, specially related to http://drupal.org/coding-standards

avpaderno’s picture

Status: Postponed (maintainer needs more info) » Needs work

I am changing status as per previous comment.

avpaderno’s picture

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

I am closing this application due to lack of replies.

avpaderno’s picture

Component: Miscellaneous » new project application
Issue summary: View changes

Please read the following links as this is very important information about CVS applications.