Skip to content

[#895] Show "Open" for stories without deadlines#897

Merged
realproject7 merged 1 commit intomainfrom
task/895-fix-deadline-open-label
Apr 20, 2026
Merged

[#895] Show "Open" for stories without deadlines#897
realproject7 merged 1 commit intomainfrom
task/895-fix-deadline-open-label

Conversation

@realproject7
Copy link
Copy Markdown
Owner

Summary

  • Deadline stat box now shows "Open" for stories with has_deadline=false instead of ambiguous "—"
  • Stories with has_deadline=true but missing last_plot_time still show "—" (rare data gap)
  • Existing behavior unchanged: sunset stories show "Complete", deadline stories show countdown

Files Changed

  • src/app/story/[storylineId]/page.tsx — split fallback branch into two cases
  • package.json — patch bump 0.1.25 → 0.1.26

Test plan

  • Open a story with has_deadline=false (e.g. The Holloway Manuscript [P6-4] SDK — @plotlink/sdk #35) — verify "Open" label
  • Open a story with a deadline — verify countdown still works
  • Open a sunset story — verify "Complete" still shows

Fixes #895

🤖 Generated with Claude Code

Separate the deadline stat box fallback into two cases:
- has_deadline=false → "Open" (intentional, no deadline set)
- has_deadline=true but missing last_plot_time → "—" (data gap)

Fixes #895

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@vercel
Copy link
Copy Markdown

vercel Bot commented Apr 20, 2026

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

1 Skipped Deployment
Project Deployment Actions Updated (UTC)
plotlink Ignored Ignored Apr 20, 2026 11:03pm

Request Review

Copy link
Copy Markdown
Collaborator

@project7-interns project7-interns left a comment

Choose a reason for hiding this comment

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

Verdict: APPROVE

Summary

The change is minimal and matches the issue acceptance criteria. Open-ended stories now show a clear Open label while the existing deadline and sunset branches remain intact.

Findings

  • No blocking findings.

Decision

Approving because the fallback branch is now correctly split between has_deadline=false and the rare missing-data case, with no evident regression in the surrounding logic.

@realproject7 realproject7 merged commit d756026 into main Apr 20, 2026
4 checks passed
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.

Deadline stat box shows '—' instead of 'No deadline' for open-ended stories

2 participants