Skip to content

Fix championship race section visibility from day 14#84

Open
dai wants to merge 2 commits into
mainfrom
codex/fix-changes-not-reflecting-after-merging-pr83
Open

Fix championship race section visibility from day 14#84
dai wants to merge 2 commits into
mainfrom
codex/fix-changes-not-reflecting-after-merging-pr83

Conversation

@dai

@dai dai commented May 23, 2026

Copy link
Copy Markdown
Owner

Motivation

  • The championship table added for Day 14 in a prior change did not appear when Day 14 was known via today data but not yet published in resultDays, because currentDay only considered published result days.

Description

  • Update app/page.tsx to compute latestKnownDay = MAY2026_TORIKUMI_DATA.today?.makuuchi.day || 0 and set currentDay = Math.max(latestPublishedResultDay, latestKnownDay) so the championship section shows once the known day reaches 14 even before results are published.

Testing

  • Ran npm test -- app/page.test.tsx which passed, and note that an earlier attempt npm test -- app/page.test.tsx --runInBand failed because Vitest does not recognize the --runInBand option.

Codex Task

@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented May 23, 2026

Copy link
Copy Markdown

Deploying o-sumo with  Cloudflare Pages  Cloudflare Pages

Latest commit: de4f58b
Status: ✅  Deploy successful!
Preview URL: https://8705bd20.o-sumo.pages.dev
Branch Preview URL: https://codex-fix-changes-not-reflec.o-sumo.pages.dev

View logs

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant