Fix Streak Calculation to Use Effective Today Date

[tmdeveloper007]

Description

Fixed calculateStreak in lib/calculate.ts to use effectiveTodayStr for todayIndex calculation instead of localTodayStr. This ensures streak calculation respects user timezone settings.

Pillar

• Other (Bug fix, refactoring, docs)

Checklist

• I have read the CONTRIBUTING.md file.
• My commits follow the Conventional Commits format.

Note

Please assign this PR to the tmdeveloper007 account.

Closes #8757

[retenta-bot]

This pull request addresses an important bug fix by ensuring that the streak calculation respects user timezone settings. It is similar to previous decisions made regarding the use of effectiveTodayStr for date calculations, which have been validated to improve accuracy in streak tracking. Thank you for your contribution, and please let me know if you need any further assistance!

[github-actions]

📦 Next.js Bundle Size Report (Gzipped Sizes)

✨ No significant bundle size changes detected.

📊 Summary of Totals

Category	PR Size	Base Size	Difference
Total JS	4939.27 KB	4939.27 KB	0 B
Total CSS	348.46 KB	348.46 KB	0 B

[Aamod007]

Nice fix! Replacing \localTodayStr\ with \effectiveTodayStr\ in the \ odayIndex\ calculation of \lib/calculate.ts\ properly aligns the streak calculation with the user's timezone settings rather than defaulting to the server's or viewer's local time. This is a crucial fix for accuracy. Good job!