feat: implement patch#41
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Caution Review failedPull request was closed or merged during review 📝 WalkthroughWalkthroughThis PR introduces five major features: leaderboard period filtering (week/month/year/all), post engagement email notifications for reactions/comments/replies, an admin staff dashboard with metrics and activity charts, new "dominator" and "survivor" badges with end-of-game evaluation, and updates to app release metadata for v26.05.06. ChangesLeaderboard Period Support
Post Engagement Email Notifications
Admin Dashboard Staff Overview
Badge Enhancements
Release & Metadata Updates
Sequence Diagram(s)sequenceDiagram
participant User as Client User
participant Web as Web App
participant API as Backend API
participant DB as Database
participant Mail as Mail Service
Note over User,Mail: Post Engagement Email Notification Flow
User->>Web: Submit reaction/comment/reply
Web->>API: POST /reactions or /comments
API->>DB: Create reaction or comment in transaction
DB-->>API: Return created record with IDs
API->>API: Check dedupe (email notification within 15m)
alt Dedupe miss (first notification)
API->>DB: Fetch post & recipient user
API->>DB: Fetch recipient email
alt Email verified & not self-notification
API->>Mail: sendReactionEmail / sendCommentEmail / sendReplyEmail
Mail-->>API: Email sent
API->>DB: Record notification event (reactionEmailNotification / commentEmailNotification / replyEmailNotification)
end
end
API-->>Web: Success response
Web-->>User: Display feedback
Note over User,Mail: Admin Dashboard Overview Load
User->>Web: Navigate to /dashboard (staff)
Web->>API: GET orpc.admin.getOverview
API->>DB: Parallel batch queries (totals, 7d deltas, top creators/players, recent games, 14d activity, jobs, roles)
DB-->>API: Aggregated results
API-->>Web: GetAdminOverviewOutputType
Web->>Web: Render totals, activity chart, tables, job status
Web-->>User: Display dashboard
Estimated code review effort🎯 4 (Complex) | ⏱️ ~60 minutes Possibly related PRs
Suggested labels
Poem
🚥 Pre-merge checks | ✅ 3 | ❌ 2❌ Failed checks (1 warning, 1 inconclusive)
✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Summary by CodeRabbit
New Features
Chores