feat: add last synced timestamp to dashboard header (#186)#1298
feat: add last synced timestamp to dashboard header (#186)#1298Tannuu18 wants to merge 1 commit into
Conversation
|
@Tannuu18 is attempting to deploy a commit to the PRIYANSHU DOSHI's projects Team on Vercel. A member of the Team first needs to authorize it. |
GSSoC Label Checklist 🏷️@Priyanshu-byte-coder — please apply the appropriate labels before merging: Difficulty (pick one):
Quality (optional):
Validation (required to score):
|
|
heyy @Priyanshu-byte-coder can you please review and merge the issue?? |
|
Thanks @Tannuu18 for the dashboard timestamp feature! Before we can merge, we need clarification on a key architectural change: This PR replaces
Also, the |
Priyanshu-byte-coder
left a comment
There was a problem hiding this comment.
This PR has 10 merge conflicts with current main (in src/app/dashboard/page.tsx, src/components/DashboardHeader.tsx, and src/lib/supabase.ts which have all been significantly updated). Please rebase onto current main and resolve the conflicts.
|
Your PR has conflicts with the current |
Shows relative time since last data sync (e.g. 'Synced 2 min ago') in the dashboard header, using a fetch-intercepting context provider. Co-authored-by: Tannuu18 <adhikaritani901@gmail.com>
Priyanshu-byte-coder
left a comment
There was a problem hiding this comment.
This PR now has merge conflicts with main due to recent merges. Please rebase onto the latest main branch and resolve all conflicts to proceed with merging.
|
This PR has merge conflicts with |
|
This PR has merge conflicts with the current |
45f63c8 to
1ac414e
Compare
3.[supabase.ts] needed the merge because the timestamp change also touched the public-profile lookup path. The safe merged version keeps Supabase from being initialized when env vars are missing or placeholder values, and it avoids build/runtime crashes in environments where Supabase is not configured. It also preserves the exact username lookup used by the public profile checks. |
Summary
Brief description of what this PR does.
Adds a global dashboard sync timestamp in the header so users can see how fresh their metrics are.
Closes #186
Type of Change
Changes Made
How to Test
Steps for the reviewer to verify this works:
Screenshots (if UI change)
Checklist
npm run lintpasses locallynpm run type-check)