[#569] Redesign profile page as social credibility trust dashboard#573
[#569] Redesign profile page as social credibility trust dashboard#573realproject7 merged 4 commits intomainfrom
Conversation
- Larger avatar (72px) with border - Trust dashboard grid: Farcaster card (username, FID, followers, Power/Pro badges), X/Twitter card (handle, followers, verified badge), Quotient Score card (score + rank), Wallet card (address + royalties) - Graceful degradation: each card only renders when data exists, wallet card always shown, wallet-only users get clean minimal profile - Mobile responsive: single column on mobile, 2-col grid on sm+ - Terminal/monospace aesthetic for stats (font-mono on numbers) - All data from existing users table — no new API calls Fixes #569 Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub. |
project7-interns
left a comment
There was a problem hiding this comment.
REQUEST CHANGES — one finding.
Farcaster profile link uses farcaster.xyz but per project conventions the correct domain is farcaster.com (not warpcast.com, not farcaster.xyz). Please update the link at the Farcaster card username href.
Otherwise the PR is clean: graceful degradation across all four data scenarios, responsive grid, terminal aesthetic on stats, no new API calls. Will approve after the URL fix.
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
project7-interns
left a comment
There was a problem hiding this comment.
T2b approves. Farcaster link corrected to farcaster.com. Trust dashboard is clean with graceful degradation across all data scenarios, responsive grid, and terminal aesthetic.
project7-interns
left a comment
There was a problem hiding this comment.
Verdict: REQUEST CHANGES
Summary
The trust dashboard redesign is close, but the wallet identity section still misses one of the explicit issue requirements.
Findings
- [medium] The wallet card never exposes a copy affordance for the connected address, so the PR does not satisfy issue #569's
truncated + copywallet identity requirement.- File:
src/app/profile/[address]/page.tsx:381 - Suggestion: add a small copy-to-clipboard control next to the truncated address while keeping the explorer link.
- File:
Decision
Requesting changes because the wallet identity acceptance criteria are still incomplete.
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
project7-interns
left a comment
There was a problem hiding this comment.
Verdict: REQUEST CHANGES
Summary
The wallet copy affordance is fixed, but the Farcaster dashboard can still render from live lookup data instead of the cached users row, which leaves the trust metrics inconsistent.
Findings
- [medium]
ProfilePagestill requestsgetFarcasterProfile(address), and that helper falls back tolookupByAddress(address)when there is no cached FID row. The header then treats any returnedfcProfileas sufficient to render the Farcaster card, but follower/following counts and badges still come fromdbUserwith0/nullfallbacks. For an address that resolves via the live Farcaster lookup but has no localusersrow yet, the new trust dashboard will show a Farcaster identity with misleading0followers /0following and no badges. That violates issue #569'susers-table-only data-source requirement and the intentional degradation requirement.- File:
src/app/profile/[address]/page.tsx:39 - File:
src/app/profile/[address]/page.tsx:208 - File:
lib/actions.ts:29 - Suggestion: derive Farcaster-card presence and fields strictly from
dbUser(fid,username,display_name,pfp_url, etc.), or suppress the card entirely when the cachedusersrow is absent instead of falling back to live lookup data.
- File:
Decision
Requesting changes because the social-credibility dashboard can still display misleading trust data for uncached Farcaster addresses.
hasFarcaster now checks dbUser.fid + dbUser.username instead of fcProfile. FC card uses dbUser fields so follower counts come from cached data, not the live API which doesn't return stats. Prevents misleading "0 followers" when user is uncached. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
project7-interns
left a comment
There was a problem hiding this comment.
Verdict: APPROVE
Summary
The redesign now satisfies the remaining review findings. The wallet identity includes the required copy affordance, and the Farcaster trust card only renders from cached users-table data.
Findings
- No remaining blocking issues found.
Decision
Approving because the social-credibility dashboard now degrades intentionally and no longer shows misleading Farcaster trust metrics for uncached addresses.
Summary
userstable — no new API callsFixes #569
Test plan
npm run build)🤖 Generated with Claude Code