Skip to content

[#187] Plot detail pages: genesis redirect#228

Merged
realproject7 merged 1 commit intomainfrom
task/187-plot-detail-polish
Mar 16, 2026
Merged

[#187] Plot detail pages: genesis redirect#228
realproject7 merged 1 commit intomainfrom
task/187-plot-detail-polish

Conversation

@realproject7
Copy link
Copy Markdown
Owner

Summary

Test plan

  • /story/2/0 redirects to /story/2
  • /story/2/1 renders chapter detail page
  • Prev/Next buttons work across chapters
  • Mobile responsive

Fixes #187

🤖 Generated with Claude Code

/story/[id]/0 now redirects to /story/[id] (genesis is displayed
on the main story page). All other acceptance criteria from #187
were already implemented in PR #227.

Fixes #187

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
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.

T2b: APPROVE — Correct genesis redirect. Plot 0 goes to main story page where genesis is displayed prominently.

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 remaining change for #187 is appropriately minimal: /story/[storylineId]/0 now redirects to the main story page, which matches the current story layout where genesis is rendered on /story/[storylineId]. The rest of the plot-detail route work already present from #227 remains intact, and the updated head passes GitHub lint-and-typecheck.

Findings

  • None.

Decision

Approving because the genesis redirect closes the remaining navigation gap without introducing scope creep.

@realproject7 realproject7 merged commit 8fd5ebb into main Mar 16, 2026
1 check 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.

[P9-4] Plot detail pages with prev/next navigation

2 participants