Skip to content

docs: point the property post's dashboard links at main - #1485

Merged
MaxGhenis merged 1 commit into
mainfrom
seat/property-links-to-main
Oct 28, 2026
Merged

MaxGhenis merged 1 commit into
mainfrom
seat/property-links-to-main

Conversation

@MaxGhenis

Copy link
Copy Markdown
Contributor

Point the property post's dashboard links at main

This is Max's condition 2 for the property post (#1460): once dashboard #290 merged, the reader-facing source links move from the pinned commit 637bb26c to dashboard main.

Why now:

  • Dashboard Create Donate page #290 merged at 22:01:45Z on 28 October as 57095fde.
  • Production now serves manifest revision 17. It approves property artifact 7bf77b35e33233e0 (vahid-ahmadi, 18:29:07Z), and its revenue figures match the post: 0, 399, 412, 429, 446 and 459 £m.
  • The artifact folder on dashboard main is byte-identical to 637bb26c. So are official_costings.json, AB2026-T6.1.csv and calculators.py.

What changes:

  • Article line 58: the "property-only microsimulation results" link now points to tree/main/…/7bf77b35e33233e0.
  • property-population/README.md:
    • lines 31 to 35 now link to main;
    • line 3 records the merge commit and says why the links moved.
  • summary.json: unchanged. It keeps 637bb26c as the immutable provenance record.

All five new links return 200. No text, figure or table changes.

🤖 Generated with Claude Code

Dashboard PR #290 merged as 57095fde with the same approved property
artifact (7bf77b35, manifest revision 17), which production now serves.
The article and the property-population README link to main; summary.json
keeps the immutable snapshot commit as provenance.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@MaxGhenis
MaxGhenis requested a review from hua7450 October 28, 2026 22:05
@vercel

vercel Bot commented Oct 28, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
policyengine-calculator-next Ready Ready Preview Oct 28, 2026 10:05pm UTC
policyengine-website Ready Ready Preview Oct 28, 2026 10:05pm UTC

Request Review

@MaxGhenis

Copy link
Copy Markdown
Contributor Author

Merging at ce7ea8d5: Max's condition 2 for the property post

This carries out the second of Max's conditions for #1460 (comment 6478406152): after dashboard #290 merged, move the source links from 637bb26c to dashboard main. The Budget coordinator confirmed this PR falls under that condition list.

Gates at this head:

  • gh pr checks exits 0.
  • MERGEABLE, not a draft, and no CHANGES_REQUESTED.
  • An independent review (GPT-6.1 Sol) said APPROVE:
    • only the six link targets and the README line 3 sentence change;
    • every path is kept, with only 637bb26c… replaced by main;
    • Create Donate page #290 merged as 57095fde with an identical artifact folder;
    • the manifest approves 7bf77b35.
  • Production's manifest (revision 17) approves property artifact 7bf77b35e33233e0, and its revenue figures match the post.

The PR is under Max's account, so he can't approve it himself, and Ziming hasn't reviewed yet. It's admin-merged with --match-head-commit ce7ea8d5, which is allowed only because every check above passes.

🤖 Generated with Claude Code

@MaxGhenis
MaxGhenis merged commit 81eb807 into main Oct 28, 2026
10 checks passed
@MaxGhenis
MaxGhenis deleted the seat/property-links-to-main branch October 28, 2026 22:12

This branch was successfully deployed

2 active deployments
Preview – policyengine-calculator-next — ce7ea8d5 Deployed Oct 28, 2026 by vercel[bot]
Preview – policyengine-website — ce7ea8d5 Deployed Oct 28, 2026 by vercel[bot]
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