New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Handling timezone aware jobs #1027

Merged

EmanElsaban merged 1 commit into master from u/emanelsabban/TRON-2354

Jan 24, 2025

Contributor

EmanElsaban commented Jan 24, 2025 •

edited

Loading

This handles the case where a datetime object is timezone aware

Testing

Installed the debian package on infrastage and observed the jobs schedule. They are scheduled correctly on their expected time.
Inspected the code manually for timezone values to see if they make sense


          Handling timezone aware jobs

0a7b434

EmanElsaban requested a review from a team as a code owner

January 24, 2025 17:43

EmanElsaban requested review from nemacysts and KaspariK

January 24, 2025 17:54

KaspariK approved these changes

View reviewed changes

nemacysts approved these changes

View reviewed changes

EmanElsaban merged commit 6dd708a into master

4 checks passed

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

Labels

None yet