Skip to content

Repoint weekly search-query report to the Gap Reports Notion page#143

Merged
jpr5 merged 1 commit into
mainfrom
chore/repoint-weekly-report-to-gap-reports
Jul 22, 2026
Merged

Repoint weekly search-query report to the Gap Reports Notion page#143
jpr5 merged 1 commit into
mainfrom
chore/repoint-weekly-report-to-gap-reports

Conversation

@jpr5

@jpr5 jpr5 commented Jul 22, 2026

Copy link
Copy Markdown
Contributor

What

Consolidates the weekly and monthly reports onto a single Notion destination — the Gap Reports page (3793aa38-1852-80a5-89d3-c3d37147aa22). The weekly search-query report previously published to the old Analytics/pathfinder page (3373aa38-1852-8152-a10b-e4aa1b8a667e); it now lands on the same Gap Reports page the monthly gap-analysis already uses.

The monthly gap-analysis workflow is untouched (it already pointed at Gap Reports).

Changed locations

  1. .github/workflows/weekly-search-report.yml (line 80) — NOTION_PARENT_PAGE_ID env → Gap Reports id (adjacent comment updated to match).
  2. scripts/weekly-search-report/weekly-search-report.ts (line 938) — DEFAULT_PARENT_PAGE_ID fallback → Gap Reports id.

Verification

Old id — zero hits across the whole repo:

$ git grep -n "3373aa38-1852-8152-a10b-e4aa1b8a667e"
(no output — exit 1)

New id — both weekly locations now present alongside the pre-existing monthly ones:

$ git grep -n "3793aa38-1852-80a5-89d3-c3d37147aa22"
.github/workflows/monthly-gap-analysis.yml:104:          NOTION_PARENT_PAGE_ID: "3793aa38-1852-80a5-89d3-c3d37147aa22"
.github/workflows/weekly-search-report.yml:80:          NOTION_PARENT_PAGE_ID: "3793aa38-1852-80a5-89d3-c3d37147aa22"
scripts/gap-analysis/README.md:71:(`3793aa38-1852-80a5-89d3-c3d37147aa22`) and can be overridden with the
scripts/gap-analysis/monthly-gap-analysis.ts:126:  process.env.NOTION_PARENT_PAGE_ID ?? "3793aa38-1852-80a5-89d3-c3d37147aa22";
scripts/weekly-search-report/weekly-search-report.ts:938:const DEFAULT_PARENT_PAGE_ID = "3793aa38-1852-80a5-89d3-c3d37147aa22";

The weekly script has no --dry-run mode, so no live run was performed (that would post to Slack #engr and publish a real Notion page). Verification is grep/inspection-based.

Quality gate: prettier clean on the edited TS file, tsc --noEmit clean, full test suite green (3554 passed / 1 skipped).

Consolidate the weekly and monthly reports onto the single Gap Reports
page (3793aa38) instead of the old Analytics/pathfinder page.
@jpr5
jpr5 merged commit 0f19f92 into main Jul 22, 2026
7 checks passed
@jpr5
jpr5 deleted the chore/repoint-weekly-report-to-gap-reports branch July 22, 2026 17:38
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