Remove placeholder news post and expand the launch post - #6
Merged
Merged
Conversation
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>
Deploying with
|
| 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 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
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/_redirectsand into a newsrc/redirects.jsin the Worker. I tested the live site before applying that, and all of them already work:To confirm those come from
static/_redirectsrather than some other rule:/wp-content/uploads/x.jpgreturned 404 while/wp-admin/setup.phpreturned 301, which matches that file's contents exactly, since it had a/wp-admin/*rule and no/wp-content/*rule./_redirectsitself returns 404 because Workers static assets consumes it as configuration instead of serving it. Wrangler also logsParsed 12 valid redirect ruleson startup, one per line in the file.So
_redirectsis 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
/donateand/aboutreturned 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/healthlisting binding names. It was returningenvKeysto 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 --checkon 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 ofsitemap.xml. The launch post page went from 27 words of body text to 566 words.Audit items left for a human
TODOcomments inwoodshop.md,metal-shop.mdandclasses.mdsay the tool lists are a reasonable guess. Someone who knows the shop should confirm them.http://; that is a Givebutter dashboard setting./donatestill wants running.🤖 Generated with Claude Code