Skip to content

Remove IP Risk database documentation - #1765

Merged
horgh merged 1 commit into
mainfrom
greg/remove-ip-risk-docs
Oct 5, 2026
Merged

horgh merged 1 commit into
mainfrom
greg/remove-ip-risk-docs

Conversation

@oschwald

@oschwald oschwald commented Oct 1, 2026 •

Copy link
Copy Markdown
Member

Removes the IP Risk database documentation:

  • content/geoip/docs/databases/ip-risk/ (the overview and binary pages)
  • the "IP Risk" entry in the GeoIP navigation menu (hugo.toml)
  • static/examples/GeoIP2-IP-Risk-CSV_Example.zip and its legacy redirect in static/_redirects
  • the matching .prettierignore entries

Kept: the release notes, the minFraud IP risk score content, and the ip_risk_snapshot field in the web service docs. Those describe other products and fields, or record past releases.

The removed pages now return 404. No redirect is added. Nothing else on the site links to them.

Tested: the Hugo build succeeds, precious lint is clean, and the link checker (mise run check-links) reports 0 errors.

🤖 Generated with Claude Code

Summary by CodeRabbit

  • Documentation
    • Removed the GeoIP IP Risk database overview and binary field reference pages, along with the IP Risk entry in the documentation menu.
    • Removed the permanent redirect for the legacy IP Risk CSV example ZIP URL. Other example ZIP redirects remain.

@coderabbitai

coderabbitai Bot commented Oct 1, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

Navigate logical layers of code changes, visualize relationships, and explore their blast radius.

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Advanced

Run ID: 359e16a4-361b-429c-b13d-d827f71b899f

📥 Commits

Reviewing files that changed from the base of the PR and between 835ec81 and ad47202.

⛔ Files ignored due to path filters (1)
  • static/examples/GeoIP2-IP-Risk-CSV_Example.zip is excluded by !**/*.zip
📒 Files selected for processing (5)
  • .prettierignore
  • content/geoip/docs/databases/ip-risk/_index.md
  • content/geoip/docs/databases/ip-risk/binary.md
  • hugo.toml
  • static/_redirects
💤 Files with no reviewable changes (5)
  • content/geoip/docs/databases/ip-risk/_index.md
  • .prettierignore
  • content/geoip/docs/databases/ip-risk/binary.md
  • hugo.toml
  • static/_redirects

Included review availability: This review used your included allowance. Your plan provides up to 1 included review per hour; 0 remain after this review.


📝 Walkthrough

Walkthrough

The IP Risk database documentation pages were deleted. The navigation entry, Prettier ignore entries, and redirect for the CSV example ZIP were also removed.

Changes

IP Risk documentation removal

Layer / File(s) Summary
Remove IP Risk documentation and related entries
content/geoip/docs/databases/ip-risk/*, hugo.toml, .prettierignore, static/_redirects
The IP Risk database overview and binary field documentation were deleted. The navigation entry, formatting ignore entries, and CSV example ZIP redirect were removed.

Priority: ⬇️ Low

Estimated code review effort: 1 (Trivial) | ~5 minutes

Change: Other

Merge Risk: ⚪ Minimal · up to ad472

This removes the IP Risk database documentation. The removed pages will return 404 with no replacement redirect, which the author states is intended. No merge-blocking risk was identified.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and concisely describes the primary change: removal of the IP Risk database documentation.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR
  • Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Autopilot is currently an internal CodeRabbit preview.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

A rabbit reads the pages gone
Then hops past menus, one by one
No more old links to follow through
The IP Risk docs bid adieu
It nibbles clover in the sun

Comment @coderabbitai help to get the list of available commands.

@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Oct 1, 2026 •

Copy link
Copy Markdown

Deploying dev-site with  Cloudflare Pages  Cloudflare Pages

Latest commit: ad47202
Status: ✅  Deploy successful!
Preview URL: https://fa9e2e28.dev-site-4ua.pages.dev
Branch Preview URL: https://greg-remove-ip-risk-docs.dev-site-4ua.pages.dev

View logs

Remove the IP Risk database pages, the example CSV download, and the
navigation entry, along with the matching .prettierignore and redirect
entries.

Keep the release notes, the minFraud IP risk score content, and the
ip_risk_snapshot web service field documentation. They describe
different products and fields, or record past releases.

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
@oschwald
oschwald force-pushed the greg/remove-ip-risk-docs branch from 0b9e9d3 to ad47202 Compare October 2, 2026 14:04
@oschwald
oschwald marked this pull request as ready for review October 2, 2026 14:04
Copilot AI balanced review requested due to automatic review settings October 2, 2026 14:04

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@horgh
horgh merged commit 4b5afe8 into main Oct 5, 2026
11 checks passed
@horgh
horgh deleted the greg/remove-ip-risk-docs branch October 5, 2026 18:50
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.

3 participants