test(api-user-details): Enhance Timezone Normalization and Calendar Data Tests#8109
Conversation
|
@Rakshak05 is attempting to deploy a commit to the jhasourav07's projects Team on Vercel. A member of the Team first needs to authorize it. |
|
This PR introduces important tests for timezone normalization and calendar data boundary alignment, which are crucial for maintaining accurate date calculations across different time zones. This aligns with previous decisions made to ensure robust handling of timezone functionalities. Thank you for your contributions, and I look forward to your updates! |
📦 Next.js Bundle Size Report (Gzipped Sizes)✨ No significant bundle size changes detected. 📊 Summary of Totals
|
Aamod007
left a comment
There was a problem hiding this comment.
Well done on enhancing the timezone testing for user details. The checks across multiple timezones using convertLocalToUtc are robust. However, this PR is blocked by a failing Vercel build check. Please resolve the issues and rerun the CI pipeline.
Aamod007
left a comment
There was a problem hiding this comment.
Upon re-evaluation, I realized I previously misinterpreted the Vercel deployment failure as an issue with this PR. The Vercel build failure seems unrelated to the changes introduced here, so I will ignore it moving forward. The rest of the PR looks good, so I'm approving it!
|
🎉 Congratulations @Rakshak05! Your PR has been successfully merged. 🚀 Thank you for contributing to CommitPulse. Your work helps us build a better tool for the community.
Keep building! 💻✨ |
Description
Fixes #6815
Pillar
Visual Preview
N/A (This PR introduces back-end unit and integration tests only).
Checklist before requesting a review:
CONTRIBUTING.mdfile.localhost:3000/api/streak?user=YOUR_USERNAME).npm run formatandnpm run lintlocally and resolved all errors (CI will fail otherwise).feat(themes): ...,fix(calculate): ...).README.mdif I added a new theme or URL parameter.