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

Comments

jenlampton’s picture

jenlampton
she / her
commented
Status: Active » Needs review
StatusFileSize
new move_settings_page-1312634-1.patch422 bytes

patch attached.

andrewmacpherson’s picture

andrewmacpherson commented
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

andrewmacpherson commented
Issue summary: View changes

updating issue link

valic’s picture

valic
Primary language Croatian
Location Osijek
commented
Issue summary: View changes
Status: Needs work » Closed (outdated)

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.