Skip to content

[#357] Hide 'Read the first Plot' button when only genesis exists#363

Merged
realproject7 merged 1 commit intomainfrom
task/357-read-plot-inactive
Mar 19, 2026
Merged

[#357] Hide 'Read the first Plot' button when only genesis exists#363
realproject7 merged 1 commit intomainfrom
task/357-read-plot-inactive

Conversation

@realproject7
Copy link
Copy Markdown
Owner

Summary

  • Button was displayed even when no chapters exist (only genesis), linking to #genesis — a useless self-link
  • Fix: only render the button when chapters.length > 0

Fixes #357

Files changed

  • src/app/story/[storylineId]/page.tsx — conditional render of the CTA button

Test plan

  • next build passes
  • Story with only genesis: button hidden
  • Story with chapters: button shown, links to /story/{id}/1

🤖 Generated with Claude Code

Button was shown even with no chapters, linking to #genesis (useless
self-link). Now only rendered when chapters.length > 0.

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.

LGTM. Correctly hides the button when only genesis exists — no more useless #genesis self-link. Clean conditional render. Approved.

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 PR matches issue #357 and removes the useless #genesis self-link by only rendering the CTA when chapters actually exist. The change is minimal and keeps the story page behavior consistent with the intended navigation model.

Findings

  • No correctness or design issues found in the submitted diff.

Decision

Approved on code review. GitHub checks were still in progress at the time of review, so merge should still wait for required checks.

@realproject7 realproject7 merged commit 18f262e into main Mar 19, 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.

[Bug] Disable 'Read the first Plot' button when only genesis exists

2 participants