Skip to content

[#112] Add inline ConnectWallet CTA to wallet-gated pages#122

Merged
realproject7 merged 1 commit intomainfrom
task/112-inline-connect-wallet
Mar 15, 2026
Merged

[#112] Add inline ConnectWallet CTA to wallet-gated pages#122
realproject7 merged 1 commit intomainfrom
task/112-inline-connect-wallet

Conversation

@realproject7
Copy link
Copy Markdown
Owner

Summary

  • Added <ConnectWallet /> button inline below the "Connect your wallet to..." prompt on 4 wallet-gated pages: /create, /chain, /dashboard/reader, /dashboard/writer
  • Users now have a clear CTA instead of a dead-end text prompt

Fixes #112

Test plan

  • Visit /create, /chain, /dashboard/reader, /dashboard/writer while disconnected — each should show the connect wallet button below the prompt text
  • Click the connect button on each page — wallet connection flow should trigger
  • After connecting, each page should render its normal content

🤖 Generated with Claude Code

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 Review: APPROVE

Clean, consistent change — <ConnectWallet /> added to all 4 wallet-gated pages with correct imports. Placement is logical (sibling to the prompt text inside flex containers with gap). No unrelated changes, no duplicate imports. +8/-0.

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 restores an actionable inline <ConnectWallet /> CTA on all four wallet-gated pages called out in issue #112. The component is reusable outside the navbar, so this resolves the dead-end prompt without adding extra complexity.

Findings

  • None.

Decision

Approve because the implementation matches the acceptance criteria, stays focused on the affected pages, and the lint-and-typecheck check passed.

@realproject7 realproject7 merged commit bf287b2 into main Mar 15, 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] ConnectWallet dead ends — inline CTA removed from pages

2 participants