Skip to content

Remove placeholder news post and expand the launch post - #6

Merged
adman234 merged 1 commit into
mainfrom
ad-grants-content-fixes
Sep 24, 2026
Merged

adman234 merged 1 commit into
mainfrom
ad-grants-content-fixes

Conversation

@adman234

Copy link
Copy Markdown
Contributor

Addresses the content findings from an Ad Grants policy audit. The audit's main finding turned out to be wrong, so this PR is smaller than the patch it came from. Details below.

The redirects are not broken

The audit reported that every old WordPress URL returns 404, and proposed moving the redirect table out of static/_redirects and into a new src/redirects.js in the Worker. I tested the live site before applying that, and all of them already work:

/about-cgw/                       301 -> /about/
/donation-membership/             301 -> /donate/
/how-to-get-involved/             301 -> /visit/
/2026/04/08/kopn-tour-april-23/   301 -> /news/kopn-tour-april-23/
/feed/                            301 -> /news/index.xml
/wp-login.php                     301 -> /
/wp-admin/setup.php               301 -> /

To confirm those come from static/_redirects rather than some other rule: /wp-content/uploads/x.jpg returned 404 while /wp-admin/setup.php returned 301, which matches that file's contents exactly, since it had a /wp-admin/* rule and no /wp-content/* rule. /_redirects itself returns 404 because Workers static assets consumes it as configuration instead of serving it. Wrangler also logs Parsed 12 valid redirect rules on startup, one per line in the file.

So _redirects is honoured on the Workers deploy, and replacing it with Worker code would have swapped working declarative config for code that has to run on every unmatched request. The file's comment did say "Cloudflare Pages reads this file", which is probably what prompted the diagnosis; that comment is corrected here.

I could not reproduce a 404 on any of those paths. The audit also noted that its first fetches of /donate and /about returned the old WordPress pages, which suggests a stale cache on its side, and that would explain the 404 readings too.

What this PR does change

Deletes the January "New Website!" post. Fifteen words ending "New information and features coming soon!", which is the under-construction placeholder Google's policy names. Its URL, and the old WordPress URL pointing at it, now redirect to the September post. That redirect had to move, because /2026/01/27/hello-world/ pointed at the post being deleted.

Expands the September launch post, which was 27 words and carried the same title, so /news/ showed two identical "New Website!" headlines. It is now an overview of what the site covers, linking to tools, visit, classes, events, donate, membership and contact. Everything it claims is on the site today, including the one-time/monthly/yearly giving options and the email updates signup.

Its photo, cgw-website-screens.jpg, is not screenshots of the site: it shows the gear logo on a laptop and a phone on a desk. It had no alt text, and now has alt text describing that.

Adds /wp-content/* next to the existing /wp-admin/* rule.

Stops /api/health listing binding names. It was returning envKeys to anyone who asked, naming the Worker's secrets. The boolean map it already returns covers the endpoint's purpose, so the name list is gone.

Verification

Hugo extended 0.165.0 with --minify --gc --panicOnWarning: clean, 31 pages. htmltest: passed, 27 documents. node --check on the Worker modules: clean.

Checked after the build: the deleted post is no longer generated, nothing links to it, /news/ lists one "New Website!" entry, and it has dropped out of sitemap.xml. The launch post page went from 27 words of body text to 566 words.

Audit items left for a human

  • Several news posts run 49 to 84 words, and four homepage events say "topic to be announced". Both need facts I do not have.
  • Hidden TODO comments in woodshop.md, metal-shop.md and classes.md say the tool lists are a reasonable guess. Someone who knows the shop should confirm them.
  • The Givebutter profile reportedly still links back over http://; that is a Givebutter dashboard setting.
  • PageSpeed Insights on mobile for the homepage and /donate still wants running.

🤖 Generated with Claude Code

Two Ad Grants website-policy items, both about content rather than links.

The January "New Website!" post was fifteen words ending "New information and
features coming soon!", which is the under-construction text the policy calls
out. It is deleted, and its URL plus the old WordPress URL that pointed at it
now redirect to the September launch post.

That September post was itself only 27 words, and carried the same title, so
/news/ listed two identical headlines. It is now a real overview of what the
new site covers, linking to the tools, visit, classes, events, donate,
membership and contact pages. Its photo shows the gear logo on a laptop and a
phone rather than screenshots of the site, so the alt text now says that.

Adds /wp-content/* alongside the existing /wp-admin/* rule.

Separately, /api/health listed the Worker's binding names to anyone who asked.
The boolean map it already returns covers what the endpoint is for, so the name
list is removed.

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

Copy link
Copy Markdown

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Preview URL Updated (UTC)
✅ Deployment successful!
View logs
website 5fd30c8 Commit Preview URL

Branch Preview URL
Sep 24 2026, 12:27 PM

@adman234
adman234 merged commit bebcfb3 into main Sep 24, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant