Skip to content

Hide field graphic when ESPN reports no possession - #469

Merged
frizat82 merged 2 commits into
devfrom
fix/field-position-no-possession
Oct 5, 2026
Merged

frizat82 merged 2 commits into
devfrom
fix/field-position-no-possession

Conversation

@frizat82

@frizat82 frizat82 commented Oct 5, 2026

Copy link
Copy Markdown
Owner

Owner report: DET@CAR right after a Carolina field goal showed the ball at a stale spot with a possession arrow. Live ESPN at that moment had possession: null, yardLine: 35, down: -1 (official timeout before the kickoff). GameSituationDto.FromSituation now returns null when nobody has the ball, so the card hides the field graphic, the same path as when ESPN sends no situation. This DTO is shared by NFL and CFB. Includes a test.

🤖 Generated with Claude Code

https://claude.ai/code/session_01DAWjtJgXaMYbzZXYJjTEYs

Between plays (kickoff after a score, timeouts) ESPN sends a situation with no possession and a
stale yardLine; the card drew a ball and an away-team arrow from it. No possessor now means no
situation, so the field graphic hides until the next play (NFL and CFB share this DTO).

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01DAWjtJgXaMYbzZXYJjTEYs
@vercel

vercel Bot commented Oct 5, 2026 •

Copy link
Copy Markdown

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

Project Deployment Actions Updated
ivleague Ready Ready Preview Oct 5, 2026 3:48am UTC

@github-actions

github-actions Bot commented Oct 5, 2026 •

Copy link
Copy Markdown

Backend Unit Test Results

✔️ Tests 1222 / 1222 - passed in 31.3s
✔️ Coverage 72.21% - passed with 30% threshold
📏 3045 / 4072 lines covered 🌿 1035 / 1578 branches covered
🔍 click here for more details

✏️ updated for commit b102499

@frizat82
frizat82 merged commit eece80d into dev Oct 5, 2026
8 checks passed
@frizat82
frizat82 deleted the fix/field-position-no-possession branch October 5, 2026 03:54

This branch was successfully deployed

1 active deployment
Preview — b102499d Deployed Oct 5, 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