feat: add customizable dashboard theme presets#1445
Conversation
|
@CyraTechZenith 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):
|
There was a problem hiding this comment.
Thanks for your first PR on DevTrack! 🎉
A maintainer will review it within 48 hours. While you wait:
- Make sure CI is passing (type-check + lint)
- Double-check the PR description is filled out and the issue is linked
- Feel free to ask questions in Discussions if you need help
If you find DevTrack useful, a ⭐ star on the repo is always appreciated — it helps the project grow and attract more contributors!
Priyanshu-byte-coder
left a comment
There was a problem hiding this comment.
Migration file modification is dangerous. PR adds DROP TABLE IF EXISTS wakatime_stats CASCADE to existing migration 20260526000000_add_wakatime_stats.sql. This migration already ran in production — Supabase won't re-run it, so the DROP is dead code locally but if someone applies it manually it destroys data. Fix: create a NEW migration file with a later timestamp (e.g. 20260529000000_reset_wakatime_stats.sql) containing the DROP + CREATE. Never modify an already-committed migration.
|
Thanks for catching that. I've reverted the changes to the existing migration and removed them from this PR to avoid modifying an already-committed migration. |
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.
|
@Priyanshu-byte-coder I have resolved all merge conflicts in the PR. Kindly take a look |
|
This PR now has merge conflicts with |
Summary
Added customizable dashboard theme presets with persistent theme selection support and improved theme management UX.
Closes #252
Type of Change
Changes Made
wakatime_statsmigration to fix UUID/text mismatch issueHow to Test
Screenshots (if UI change)
Included screenshots for:
Checklist
npm run lintpasses locallynpm run test)Accessibility Checklist
Additional Notes
npm run testcurrently contains unrelated upstream failing tests (supabase-guard,github-achievements, and auth scope assertion mismatch). No new test failures were introduced by this PR.