Skip to content

v1.1.1: Fix timezone issues in monthly chart#32

Merged
laminko merged 3 commits intomainfrom
dev
Jun 16, 2025
Merged

v1.1.1: Fix timezone issues in monthly chart#32
laminko merged 3 commits intomainfrom
dev

Conversation

@laminko
Copy link
Copy Markdown
Owner

@laminko laminko commented Jun 16, 2025

Summary

  • Fix timezone conversion bug in MonthlySpendingChart where spending entries appeared on wrong day
  • Migrate from Moment.js to date-fns for better performance and smaller bundle size
  • Resolve issue where June 10th spending showed as June 11th in monthly chart

Changes

  • 🐛 Fixed: Timezone conversion in MonthlySpendingChart.vue
  • 🔄 Migrated: Moment.js → date-fns/date-fns-tz
  • 🧹 Cleaned: Removed debug logging
  • 📦 Improved: Smaller bundle size with tree-shaking

Test plan

  • Build passes without TypeScript errors
  • Monthly chart displays spending on correct days
  • UTC to local timezone conversion works properly
  • No console errors or debug logs

🤖 Generated with Claude Code

laminko and others added 3 commits June 15, 2025 16:28
Add missing categories table creation SQL including:
- Table structure with required columns
- Unique constraint on user_id + name
- Performance index on user_id
- Complete RLS policies for CRUD operations
- Updated documentation notes

Fixes #30

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-authored-by: Claude <noreply@anthropic.com>
- Migrate from moment/moment-timezone to date-fns/date-fns-tz for better tree-shaking
- Fix timezone conversion bug in MonthlySpendingChart where dates were incorrectly shifted
- Replace .toISOString().split('T')[0] with format() to avoid UTC conversion issues
- Update all date utility functions to use date-fns equivalents
- Resolve issue where spending entries appeared on wrong day in monthly chart
- Clean up debug logging code

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
@vercel
Copy link
Copy Markdown

vercel bot commented Jun 16, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
spend-check ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 16, 2025 1:12pm

@laminko laminko merged commit 80259cd into main Jun 16, 2025
3 checks passed
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