Skip to content

Add timezone offset to CronDaemon#4

Open
csim wants to merge 1 commit into
kevincolyar:masterfrom
csim:master
Open

Add timezone offset to CronDaemon#4
csim wants to merge 1 commit into
kevincolyar:masterfrom
csim:master

Conversation

@csim

@csim csim commented Jan 27, 2016

Copy link
Copy Markdown

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
Copy Markdown
Owner

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

@csim

csim commented Jan 29, 2016

Copy link
Copy Markdown
Author

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