Skip to content

[WIP][14.0][ADD] Introduce new module 'cron_dst' for handling timezones and Daylight Saving Changes for cronjobs. - #2348

Closed
manengel wants to merge 6 commits into
OCA:14.0from
initOS:14.0-cron_dst
Closed

[WIP][14.0][ADD] Introduce new module 'cron_dst' for handling timezones and Daylight Saving Changes for cronjobs.#2348
manengel wants to merge 6 commits into
OCA:14.0from
initOS:14.0-cron_dst

Conversation

@manengel

@manengel manengel commented May 27, 2022

Copy link
Copy Markdown

Hello dear contributors,
we would like to introduce the module cron_dst to the OCA.

It has been created with the purpose to add time zones as a factor to the time of execution of cronjobs and, for regions
affected by that, also ease the handling of Daylight Saving Changes and ensuring that manual adjustments
are no longer required.

We hope that the tool can be useful for many other individuals.

Possibly, it does need improvements. It has some tests.

@manengel manengel changed the title [14.0][ADD] Introduce new module 'cron_dst' for handling timezones and Daylight Saving Changes. [14.0][ADD] Introduce new module 'cron_dst' for handling timezones and Daylight Saving Changes for cronjobs. May 27, 2022
@manengel manengel changed the title [14.0][ADD] Introduce new module 'cron_dst' for handling timezones and Daylight Saving Changes for cronjobs. [WIP][14.0][ADD] Introduce new module 'cron_dst' for handling timezones and Daylight Saving Changes for cronjobs. Jun 2, 2022
@florian-dacosta

Copy link
Copy Markdown
Contributor

Hello @manengel
I think this module does the same more or less : #2289
What do you think ?

@manengel

manengel commented Jun 7, 2022

Copy link
Copy Markdown
Author

Hello @florian-dacosta,
yes, in general you are right about them being quite similar (at least in purpose). We might need to compare the both to each other more closely and then decide which PR to keep and maintain.

@thomaspaulb

Copy link
Copy Markdown
Contributor

Since it looks like the other one was already merged in 16.0, am closing this

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.

3 participants