40.01: re-point the engagement after-clock at the rebuild SHA - #497
Merged
Merged
Conversation
Contributor
|
Important
This repository does not receive automatic reviews because it has fewer than 10 stars. ⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Plus Run ID: 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. Comment |
The blog landed in three merges (f80de80 index+tags, f3f9353 Phase 0, e1fa540 the post rebuild). The ship marker still named the first, so the 28-day window would have started against a blog that was two-thirds old - exactly the error the whole-blog measurement pivot was meant to avoid. Also derives the read date from the actual deploy instead of pinning 2026-09-17: a delayed deploy shifts the window, it doesn't shorten it. Caught by a scheduled OKF maintain tick; generalised in .okf/log.md - a ship marker written mid-sprint must name the LAST merge that changes the measured thing, and the tell that it's stale is the doc still describing its scope in pre-pivot phase language. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
I told Paul twice today, and wrote into 2608 twice, that the GA4 key-event toggles were console-only and therefore his. PR #495 marked contact_cta_click through the GA4 UI, which falsifies that: Admin -> Data display -> Events -> Create event -> 'Create with code' needs no Admin API and no already-received data. Plan and README corrected, including the two dialog traps ($1 default key-event value books phantom revenue; counting method stays 'Once per event'). The failure generalises past GA4: I inferred a capability limit from the tool I reached for (read-only Data API) and reported it as a property of the task. A parked item is a gate that never opens. Still open and now correctly owned by Phase 0.1 rather than Paul: page_view is still marked a key event, so 4,063 page views read as conversions. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
pftg
force-pushed
the
blog-measure-marker
branch
from
August 20, 2026 18:24
5d46046 to
4ba7d7b
Compare
…browser' Attempted the page_view un-mark after correcting the console-only claim. The UI path from #495 is right, but chrome-devtools MCP drives a separate Chrome-for-Testing profile that is signed OUT - analytics.google.com redirects to accounts.google.com - while claude-in-chrome drives Paul's real, signed-in Chrome and is currently not connected. Recorded both channels so the next session names the actual blocker (connect the extension) instead of re-deriving either wrong conclusion. Did not sign in; entering credentials is prohibited and the block is the channel, not the task. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
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.
Docs-only. Caught by a scheduled
/okf:okf maintaintick.The blog rebuild landed in three merges today —
f80de8088(index + tagpages),
f3f9353b6(Phase 0 instrumentation + tooling),e1fa5409d(the posttemplate rebuild).
40.01's ship marker still named the first as theafter-clock trigger, so the 28-day engagement window would have started
measuring a blog that was still two-thirds old — precisely the error the
whole-blog measurement pivot was meant to avoid.
e1fa5409d(last merge that changes themeasured thing).
2026-09-17 — a delayed deploy shifts the window, it doesn't shorten it.
whole-blog rebuild", matching the pivot.
Generalised in
.okf/log.md: a ship marker written mid-sprint ages badly, andthe tell that it's stale is the doc still describing its own scope in
pre-pivot phase language.
bin/hugo-buildgreen; OKF conformant under--strict.🤖 Generated with Claude Code