As per the original post at #887268: Port GeoIP API to D7 the settings page should maybe live at admin/config/regional/language/configure/geoip now that we're in D7

CommentFileSizeAuthor
#1 move_settings_page-1312634-1.patch422 bytesjenlampton
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

jenlampton’s picture

Status: Active » Needs review
FileSize
422 bytes

patch attached.

andrewmacpherson’s picture

Status: Needs review » Needs work

Not sure why this needs to be nested under a /language/ path fragment.

How about this shorter path?
admin/config/regional/geoip

Consider adding a configure directive to the module .info file, which will appear on the modules page too.

andrewmacpherson’s picture

Issue summary: View changes

updating issue link