Skip to content

Fix activity timestamp + re-enable nav - #18

Open
snackman wants to merge 2 commits into
mainfrom
fix-activity
Open

snackman wants to merge 2 commits into
mainfrom
fix-activity

Conversation

@snackman

@snackman snackman commented May 6, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Root cause: OpenSea event_timestamp is Unix seconds, but new Date() treats it as milliseconds → dates in 1970 → immediately pruned by 90-day retention
  • Fix: Multiply by 1000 when timestamp is a number
  • Nav link will be re-added once verified on preview

Test plan

  • Hit cron endpoint with ?backfill=true on preview
  • Verify /api/activity returns events
  • Verify /activity page renders correctly
  • Re-add nav link

🤖 Generated with Claude Code

OpenSea event_timestamp is seconds since epoch, not an ISO string.
new Date(seconds) creates dates in 1970, which get immediately pruned
by the 90-day retention policy.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@vercel

vercel Bot commented May 6, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
toppings Ready Ready Preview, Comment May 6, 2026 4:23pm

Request Review

- Listings/offers use `asset` not `nft`, and event_type is "order"
  with order_type distinguishing listing vs offer
- Remove "cancel" event type (returns 400 from OpenSea)
- Use order_hash for listing/offer dedup IDs
- Re-add Activity nav link

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

1 participant