Skip to content

Add BS Pod NFL over-under tracking - #198

Merged
matsonj merged 4 commits into
masterfrom
feature/bs-pod-over-unders
Aug 28, 2026
Merged

Add BS Pod NFL over-under tracking#198
matsonj merged 4 commits into
masterfrom
feature/bs-pod-over-unders

Conversation

@matsonj

@matsonj matsonj commented Aug 28, 2026

Copy link
Copy Markdown
Owner

Summary

  • add reviewed 2026 NFC Bill Simmons and Cousin Sal picks with live settlement logic
  • add NFC summary page with projected wins, pick statuses, palette-aligned status formatting, and sidebar placement
  • add dbt source/model/docs/tests and Evidence queries

Verification

  • full pipeline passed
  • Evidence production build passed
  • browser-verified summary and team routes

@matsonj matsonj left a comment

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

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

Code review: no actionable findings. Reviewed the full diff, changed-file boundaries, query/model behavior, validation tests, Evidence page, and sidebar metadata. Removed the obsolete team-page query and unused summary/model fields during review; the final projection query also preserves NULL when simulation data is unavailable. Local validation passes: targeted dbt build (25/25) and Evidence production build. GitHub CI is still pending.

@matsonj
matsonj merged commit 961fdac into master Aug 28, 2026
1 check passed
@matsonj
matsonj deleted the feature/bs-pod-over-unders branch August 28, 2026 22:19
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