Skip to content

Fix some typos#1533

Merged
oschwald merged 22 commits intomainfrom
wstorey/tweaks
Jan 12, 2026
Merged

Fix some typos#1533
oschwald merged 22 commits intomainfrom
wstorey/tweaks

Conversation

@horgh
Copy link
Contributor

@horgh horgh commented Jan 12, 2026

No description provided.

horgh and others added 20 commits January 12, 2026 08:58
- Add missing article "a" before "Proxy Detection Legacy customer"
- Add missing word "using" before "your Sandbox account"
- Fix "rather the" to "rather than the"
- Fix duplicate "for for" to "for"

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Add missing spaces before backticks in boolean field descriptions.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
- Fix inconsistent list marker and escaped underscore in domain.md
- Fix escaped underscore in is_residential_proxy field name in 2020 release notes

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
- Fix subject-verb agreement: "follow" → "follows" in GeoIP responses
- Fix duplicate words: "to to", "for for", "for the for the", "country code country code", "ensure ensure"
- Fix missing word: "APIs see how" → "APIs to see how"
- Fix wrong word: "effect" → "affect"

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
The confidence factors release date was incorrectly written as 2025.
December 13, 2024 is a Friday (as stated), while December 13, 2025
is a Saturday.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
The <Alert type="warning"> tag was incorrectly placed inside a Hugo
{{< alert warning >}} shortcode, making it inconsistent with the
identical alert block earlier in the file.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Change [SERVICE TYPE] to [SERVICE-TYPE] for consistency with the
surrounding text which uses the hyphenated format.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
The billing section incorrectly linked to #shipping-address and the
shipping section incorrectly linked to #billing-address.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
- Change "Request API documentation" to "Response API documentation"
  (the link points to /responses)
- Change "input" to "output" (this section describes outputs)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
- Fix Anonymous Plus block filenames from Plusv4/Plusv6 to IPv4/IPv6
- Fix Enterprise locations example filename to match database name

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
- Remove extra space before period in city-and-country.md
- Fix version typo v.10 to v1.0 in legacy.md VB.NET example
- Fix double slash in CSV file path in whats-new-in-geoip2.md

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Links were incorrectly pointing to minFraud documentation instead of
GeoIP documentation for authorization section.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Remove incorrect article before uncountable noun "data".

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@cloudflare-workers-and-pages
Copy link

cloudflare-workers-and-pages bot commented Jan 12, 2026

Deploying dev-site with  Cloudflare Pages  Cloudflare Pages

Latest commit: c1695e1
Status: ✅  Deploy successful!
Preview URL: https://aee4d14b.dev-site-4ua.pages.dev
Branch Preview URL: https://wstorey-tweaks.dev-site-4ua.pages.dev

View logs

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Eight CSV files containing data location for each locale code, where
<code>{locale}</code> can be “de”, "en", “es”, “fr”, “ja”, “pt-BR”,
“ru”, and zh-CN.
"ru", and "zh-CN".
Copy link
Member

Choose a reason for hiding this comment

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

If we are changing these, we probably should change the ones on the line above. Maybe it would make sense to change all occurrences in the repo.

Copy link
Contributor Author

@horgh horgh Jan 12, 2026

Choose a reason for hiding this comment

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

Hmm, I started changing them all, but it seems like a bigger change. I wonder which way we'd want. I think I'll just revert that one above for now.

{{</ csv-example-files >}}

- [GeoIP2-Domain-CSV_Example.zip](/static/GeoIP2-Domain-CSV_Example.zip)
{{</ csv-example-files >}}
Copy link
Member

Choose a reason for hiding this comment

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

The white space change here seems wrong to me. The escaping probably doesn't matter either way.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It does look a bit wrong. It is what we do on all the other pages though. It looks like it renders okay. I'm inclined to leave it rather than update everywhere else.

the minFraud scoring engine, or for geolocation or IP intelligence. It is
designed for testing purposes only. {{</ alert >}}
but this data is not accurate and should not be used to evaluate the accuracy of
GeoIP geolocation or IP intelligence data. It is designed for testing purposes
Copy link
Member

Choose a reason for hiding this comment

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

I wonder if this change is really what we want.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I considered this. This page is specific to the GeoIP sandbox, so it made sense to me. Maybe I'll just revert and ask product people to look at it though.

Copy link
Member

Choose a reason for hiding this comment

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

Oh, I see. I missed that this was the GeoIP page. It is probably fine in that case.

@oschwald oschwald merged commit dec46f8 into main Jan 12, 2026
15 checks passed
@oschwald oschwald deleted the wstorey/tweaks branch January 12, 2026 18:51
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