Skip to content

fix: Update timezone from ist to Asia/Kolkata in quartz app#717

Merged
suvanbanerjee merged 1 commit intodevelopmentfrom
fix/firefox-timezone
Apr 10, 2026
Merged

fix: Update timezone from ist to Asia/Kolkata in quartz app#717
suvanbanerjee merged 1 commit intodevelopmentfrom
fix/firefox-timezone

Conversation

@suvanbanerjee
Copy link
Copy Markdown
Contributor

Pull Request

Description

Root cause was an invalid timezone identifier. Updated to the correct IANA timezone for India (Asia/Kolkata). Also corrected the test data, which was using UK/BST timestamps, and replaced them with IST values.

Fixes #716

How Has This Been Tested?

Edited the test timestamps to IST and ran jest

image image

Checklist:

  • My code follows OCF's coding style guidelines
  • I have performed a self-review of my own code
  • I have made corresponding changes to the documentation
  • I have added tests that prove my fix is effective or that my feature works
  • I have checked my code and corrected any misspellings

@suvanbanerjee suvanbanerjee requested a review from braddf April 8, 2026 12:59
@suvanbanerjee suvanbanerjee self-assigned this Apr 8, 2026
@vercel
Copy link
Copy Markdown

vercel bot commented Apr 8, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
quartz-app Ready Ready Preview, Comment Apr 8, 2026 0:59am
1 Skipped Deployment
Project Deployment Actions Updated (UTC)
nowcasting-app Ignored Ignored Apr 8, 2026 0:59am

Request Review

Copy link
Copy Markdown
Contributor

@braddf braddf left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Whoops, maybe this was me way back when, good work finding this bug 👍

@suvanbanerjee suvanbanerjee merged commit 2e9cd41 into development Apr 10, 2026
4 checks passed
@suvanbanerjee suvanbanerjee deleted the fix/firefox-timezone branch April 10, 2026 11:11
@braddf braddf mentioned this pull request Apr 16, 2026
8 tasks
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.

2 participants