Skip to content
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

Add timezone offset to CronDaemon #4

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

csim
Copy link

@csim csim commented Jan 27, 2016

Thanks for posting this repo, it is helpful to me when creating simple scheduled background jobs.

I made a small change to allow a timezone offset such that a job can be scheduled with a specific timezone in mind. I thought this might be helpful to others.

Cheers!

@kevincolyar
Copy link
Owner

Thanks csim. I was wondering if this should be the responsibility of the task to offset the time?

@csim
Copy link
Author

csim commented Jan 29, 2016

I guess it depends on whether ALL the tasks should use the same timezone. I suppose you could add a property to the task if you wanted tasks to be scheduled in multiple timezones. I am only interested in running tasks in one timezone.

Hangfire specifies the timezone at the task level.

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