Page MenuHomePhabricator

Translate "Connection type" data returned from MaxMind
Closed, ResolvedPublic2 Estimated Story Points

Description

IPInfo displays data from MaxMind about an IP address. Currently "Connection type" data is displayed as returned from MaxMind. We should instead display a message translation, depending on the type returned.

Possible types are:

  • Dialup
  • Cable/DSL
  • Corporate
  • Cellular

Reference: https://github.com/maxmind/GeoIP2-php/blob/7598c36acb96b18c2f30da4d03a4c340bc8fac92/src/Model/ConnectionType.php#L12
An example of where it was useful to clarify: https://gerrit.wikimedia.org/r/plugins/gitiles/mediawiki/extensions/IPInfo/+/refs/heads/master/i18n/qqq.json#29

AC

Event Timeline

Tchanders set the point value for this task to 2.
STran updated the task description. (Show Details)

Change 822497 had a related patch set uploaded (by TsepoThoabala; author: TsepoThoabala):

[mediawiki/extensions/IPInfo@master] Translate "Connection type" data returned from MaxMind

https://gerrit.wikimedia.org/r/822497

Change 822497 merged by jenkins-bot:

[mediawiki/extensions/IPInfo@master] Translate "Connection type" data returned from MaxMind

https://gerrit.wikimedia.org/r/822497

dom_walden subscribed.

I can see translations (en, fr, de) for various values of the "Connection method" (Corporate, Cellular) in the Infobox.

We still see Not available when there is no data.

Test environment: local docker IP Info 0.0.0 (669fac4) 06:29, 19 August 2022.

We still see Not available when there is no data.

Confirming that we are continuing to see Not available in the chosen language (as we already do).