PrestaShopCorp / cronjobs

Manage all your automated web tasks from a single interface
18 stars 91 forks source link

Use PHP time #40

Open vallka opened 7 years ago

vallka commented 7 years ago

If MySQL server is using different time zone, logging executed events showed incorrect time. Using PHP date() function instead MySQL NOW() solves the problem

vallka commented 5 years ago

1) if MySQL is using different time zone incorrect time is shown in Task List. Sometimes it is diffecult to change a timezone on a shared hosting 2) Day of Week is shown incorrectly in Task List

Both issues corrected

PierreRambaud commented 5 years ago

Hi and sorry for the dealy, is this pull request still relevant / needed?

vallka commented 5 years ago

Yes

------ Original Message ------ From: "GoT" notifications@github.com To: "PrestaShop/cronjobs" cronjobs@noreply.github.com Cc: "Valentine" vallka@vallka.com; "Author" author@noreply.github.com Sent: 15/10/2019 09:39:29 Subject: Re: [PrestaShop/cronjobs] Use PHP time (#40)

Hi and sorry for the dealy, is this pull request still relevant / needed?

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/PrestaShop/cronjobs/pull/40?email_source=notifications&email_token=ADBMNHKJAYP46YALCQ4L5ALQOV6UDA5CNFSM4DTSQOHKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEBH5YAA#issuecomment-542104576, or unsubscribe https://github.com/notifications/unsubscribe-auth/ADBMNHLB6R7OQBWZ2QWBRKDQOV6UDANCNFSM4DTSQOHA.

-- This email has been checked for viruses by Avast antivirus software. https://www.avast.com/antivirus

Progi1984 commented 4 years ago

@vallka Thank you for your contribution. Could you fix the conflict file with rebase ? And could you give us a scenario for testing this PR ? Thanks