Skip to content

Use 'GeoIP'/'GeoLite' branding in documentation#264

Open
oschwald wants to merge 2 commits into
mainfrom
greg/stf-555
Open

Use 'GeoIP'/'GeoLite' branding in documentation#264
oschwald wants to merge 2 commits into
mainfrom
greg/stf-555

Conversation

@oschwald
Copy link
Copy Markdown
Member

Updates prose/documentation to refer to the products as "GeoIP"/"GeoLite" instead of "GeoIP2"/"GeoLite2". Technical identifiers (packages, class names, .mmdb filenames, edition IDs, the geolite.info hostname, URLs) are unchanged.

🤖 Generated with Claude Code

MaxMind no longer ships the legacy products, so refer to the products as
'GeoIP' and 'GeoLite' rather than 'GeoIP2'/'GeoLite2' in prose. Technical
identifiers (packages, class names, filenames, edition IDs, hostnames,
URLs) are unchanged.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Copy link
Copy Markdown

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request updates a docblock comment in src/MinFraud/Model/IpAddress.php to refer to 'GeoIP' instead of 'GeoIP2'. There are no review comments, and I have no feedback to provide.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

/**
* @var IpAddress an object containing GeoIP2 and minFraud Insights
* @var IpAddress an object containing GeoIP and minFraud Insights
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

From Claude, looks right:

  - Missed prose docblock — src/MinFraud/Model/GeoIp2Location.php:10: still says "Model of the GeoIP2 Location information…". Analogous to docblocks updated in sibling files in
  this same PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants