Skip to content

Use Asia/Kolkata instead of legacy Asia/Calcutta#2559

Open
elibosley wants to merge 1 commit intomasterfrom
codex/fix/FIX
Open

Use Asia/Kolkata instead of legacy Asia/Calcutta#2559
elibosley wants to merge 1 commit intomasterfrom
codex/fix/FIX

Conversation

@elibosley
Copy link
Member

@elibosley elibosley commented Feb 26, 2026

Summary

  • replace legacy India timezone ID Asia/Calcutta with canonical Asia/Kolkata in timezone lookup key
  • updates the value used when selecting (UTC+05:30) Chennai, Kolkata, Mumbai, New Delhi

Testing

  • verified both include paths resolve to the same updated key entry (Asia/Kolkata)

Summary by CodeRabbit

  • Chores
    • Updated timezone identifiers to align with current international naming standards while preserving all UTC offset and location information.

fixes #2560

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 26, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Disabled knowledge base sources:

  • Linear integration is disabled

You can enable these sources in your CodeRabbit configuration.

📥 Commits

Reviewing files that changed from the base of the PR and between 30af957 and 1af3c7c.

📒 Files selected for processing (1)
  • emhttp/plugins/dynamix/include/timezones.key

Walkthrough

The timezone configuration file was updated to rename the timezone identifier Asia/Calcutta to Asia/Kolkata while maintaining the UTC+05:30 offset and city list. An existing Pacific timezone entry was reformatted with no net content change.

Changes

Cohort / File(s) Summary
Timezone Configuration
emhttp/plugins/dynamix/include/timezones.key
Renamed Asia/Calcutta timezone identifier to Asia/Kolkata with identical UTC offset and city list. Pacific timezone entry reformatted without functional change.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰✨ From Calcutta's name of old, now Kolkata takes the stage,
A timezone identifier updated for the modern age,
Asia's pearl now properly styled, UTC stays the same,
Time flows true as ever, just a cleaner, proper name! ⏰

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately describes the main change: replacing the legacy Asia/Calcutta timezone identifier with the canonical Asia/Kolkata identifier in the timezones.key file.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch codex/fix/FIX

Comment @coderabbitai help to get the list of available commands and usage tips.

@github-actions
Copy link

🔧 PR Test Plugin Available

A test plugin has been generated for this PR that includes the modified files.

Version: 2026.02.26.1747
Build: View Workflow Run

📥 Installation Instructions:

Install via Unraid Web UI:

  1. Go to Plugins → Install Plugin
  2. Copy and paste this URL:
https://preview.dl.unraid.net/pr-plugins/pr-2559/webgui-pr-2559.plg
  1. Click Install

Alternative: Direct Download

⚠️ Important Notes:

  • Testing only: This plugin is for testing PR changes
  • Backup included: Original files are automatically backed up
  • Easy removal: Files are restored when plugin is removed
  • Conflicts: Remove this plugin before installing production updates

📝 Modified Files:

Click to expand file list
emhttp/plugins/dynamix/include/timezones.key

🔄 To Remove:

Navigate to Plugins → Installed Plugins and remove webgui-pr-2559, or run:

plugin remove webgui-pr-2559

🤖 This comment is automatically generated and will be updated with each new push to this PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Unraid using an obsolete, legacy identifier, TZ="Asia/Calcutta" instead of TZ="Asia/Kolkata"

2 participants