Skip to content

Fix BigQuery weekly active visitor history - #4703

Open
steve8708 wants to merge 6 commits into
mainfrom
changes-7234
Open

Fix BigQuery weekly active visitor history#4703
steve8708 wants to merge 6 commits into
mainfrom
changes-7234

Conversation

@steve8708

Copy link
Copy Markdown
Contributor

Summary

  • Repair the known malformed weekly active visitor SQL on the canonical BigQuery dashboard.
  • Add a regression test for non-empty SQL with an always-true time-range branch.

Verification

  • Analytics repair spec: 21 passed
  • Analytics typecheck passed
  • pnpm guards: 71 passed
  • Beta dashboard panel repaired and rendered with historical Jun-Sep data; corrected BigQuery query returned nonzero WAU values before Sep 1.

@github-actions

github-actions Bot commented Sep 10, 2026

Copy link
Copy Markdown
Contributor

Here's a visual recap of what changed:

Visual recap

Open the full interactive recap

builder-io-integration[bot]

This comment was marked as outdated.

builder-io-integration[bot]

This comment was marked as outdated.

builder-io-integration[bot]

This comment was marked as outdated.

builder-io-integration[bot]

This comment was marked as outdated.

@builder-io-integration builder-io-integration Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Builder reviewed your changes — looks good ✅

Review Details

Incremental Code Review Summary

The latest increment adds an analytics changelog entry documenting the BigQuery active-user chart repair. The existing exact malformed-query repair and narrowly scoped QA smoke warning suppression remain unchanged from prior clean reviews. No actionable issue was identified in the available review results, and the changelog metadata/content is consistent with the analytics template changelog convention.

Risk assessment: Standard.

🧪 Browser testing: Skipped — no user-facing product UI changed; this increment adds changelog metadata only.

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