-
Notifications
You must be signed in to change notification settings - Fork 5
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.
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
Request to support running on server local time instead of UTC time zone #109
Comments
Hey Sam! 👋 The decision to use UTC everywhere was intentional and there's currently no support for other time zones. I don't know what it would take to change that approach, but I think we can differentiate between two options:
I feel like the first option is more suited. |
Ah, unfortunately, the issue we're facing is not for UI display, but rather alignment of the Jobbr scheduling with other job scheduling systems which run on CET (Central European Time) time zone. Due to Daylight Saving Time, CET can vary between UTC+1 and UTC+2. |
Seems the providers are registered and injected, as such it would be possible to use a different provider implementation. The main question is, how could this be configured and where. I'm open for suggestions, if you want to think of a design. |
Would it be that simple to just inject a time provider different from |
As for dependency injection, there's |
Currently IDateTimeProvider is only implemented by UtcNowTimeProvider and exposes method for GetUtcNow() which is used extensively in Scheduling logic. Would it be possible to extend to support running on server local time instead?
The text was updated successfully, but these errors were encountered: