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. |
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.