CVS edit link for seikan

I'm using GeoLocation technology from both Maxmind and IP2Location. Due the accuracy of the data, I need to use both database in Drupal. However, there is no IP2Location API available in Drupal. So, I developed a wrapper to use IP2Location database in Drupal.

Comments

seikan’s picture

Title: seikan [seikan] » IP2Location [seikan]
Status: Postponed (maintainer needs more info) » Needs review
StatusFileSize
new8.33 KB

IP2Location API is a Drupal module to use GeoLocation database from IP2Location.

IP2Location is a geo IP solution to help you to identify visitor's geographical location, i.e. country, region, city, latitude, longitude, ZIP code, time zone, connection speed, ISP and domain name, IDD country code, area code, weather station code and name using a proprietary IP address lookup database and technology without invading the Internet user's privacy.

avpaderno’s picture

Title: IP2Location [seikan] » seikan [seikan]

Please change only the status, when you upload new code; other metadata are not though to be changed by the applicant.

avpaderno’s picture

The previous application was #576700: seikan [seikan].

avpaderno’s picture

Status: Needs review » Needs work

As I have already reported, files available from third-party site should not be included in Drupal.org CVS.

Then, the directory data is empty; that means the directory will not be present in the archive generated by the packaging script. If you want it appears, you need to add a file in that directory (it is enough to add a file README.txt)

seikan’s picture

Status: Needs work » Needs review
StatusFileSize
new8.25 KB

I have removed the data folder.

avpaderno’s picture

Status: Needs review » Needs work

See my previous comment; you missed one point.

seikan’s picture

Status: Needs work » Needs review
StatusFileSize
new8.54 KB

Sorry about that. I have added the README.txt as well.

avpaderno’s picture

Status: Needs review » Needs work

See the first sentence I wrote in comment #4; it is not about the file README.txt, but files available from other sites.

seikan’s picture

Status: Needs work » Needs review
StatusFileSize
new3.08 KB

I have removed all third-party files. Any addition files required has been stated in README.txt now.

avpaderno’s picture

Status: Needs review » Fixed

The directory data, and lib will not appear in the archive the users will download from Drupal because they don't contain anything; you should put a README.txt so that the directory will be created in the archive.

seikan’s picture

StatusFileSize
new3.55 KB

Thanks. I have uploaded the latest package.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

avpaderno’s picture

Component: Miscellaneous » new project application
Assigned: Unassigned » avpaderno
Issue summary: View changes
Status: Closed (fixed) » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.