Skip to content

[#204] Reader dashboard with donation history#63

Merged
realproject7 merged 1 commit intomainfrom
task/204-reader-dashboard
Mar 14, 2026
Merged

[#204] Reader dashboard with donation history#63
realproject7 merged 1 commit intomainfrom
task/204-reader-dashboard

Conversation

@realproject7
Copy link
Copy Markdown
Owner

Summary

  • Create /dashboard/reader route (client component, wallet-gated)
  • Fetch donation history by connected wallet address via react-query
  • Per-donation row: storyline ID, date, $PLOT amount formatted from wei
  • Total donated amount displayed in header
  • Portfolio section (token holdings, portfolio value) marked as Phase 5 placeholder (P5-7b)
  • Wallet connection prompt when disconnected

Test plan

  • tsc --noEmit passes
  • eslint passes
  • Shows connect prompt when wallet not connected
  • Queries donations filtered by connected wallet address
  • Portfolio section clearly deferred to Phase 5
  • Amount formatting handles wei-to-ether conversion

Fixes #204

🤖 Generated with Claude Code

…olio placeholder

- Create /dashboard/reader route (client component, wallet-gated)
- Fetch donation history by connected wallet via react-query
- Show donation list with storyline ID, date, and $PLOT amount
- Display total donated amount
- Portfolio section placeholder clearly marked for Phase 5 (P5-7b)
- Terminal aesthetic throughout

Fixes #204

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.

Verdict: APPROVE

Summary

The reader dashboard matches the assigned P4-4 scope: it is wallet-gated, shows donation history for the connected wallet, and clearly defers portfolio metrics to Phase 5. The repository check is passing.

Findings

  • None.

Decision

Approving because the donation-history query is consistent with the current indexer/storage model and the page structure leaves the future portfolio area clearly marked as Phase 5 work.

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: APPROVED

Reader dashboard mirrors the writer dashboard pattern cleanly. Findings:

  1. Wallet gating — same pattern as PR #62, consistent UX.
  2. Donation query — filtered by donor_address (lowercased), ordered by recency. No hidden filter needed on donations table (correct — only storylines/plots have moderation).
  3. Total calculationBigInt accumulation with formatUnits(_, 18) for 18-decimal display. Correct approach for ERC20 amounts.
  4. Portfolio placeholder — clearly marked as Phase 5 / P5-7b. Good forward structure.
  5. DonationRow — shows storyline ID, date, and amount. Clean layout.
  6. CI green.

No issues found.

@realproject7 realproject7 merged commit cad5f1c into main Mar 14, 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.

2 participants