coopiteasy / addons

Various modules that extend or improve Odoo Community.
GNU Affero General Public License v3.0
21 stars 17 forks source link

[12.0][FIX] make company_today cron run once a day #279

Closed huguesdk closed 1 year ago

huguesdk commented 1 year ago

description

make company_today cron run once a day at 00:00:00 utc instead of every minute, to avoid recomputing fields that depend on it over and over uselessly. odoo handles all time-related data in utc, so running it more than once after midnight (utc) does not change anything.

also set the cron noupdate so it can be customized by users without being overwritten at the next update.

odoo task

t9911

codecov-commenter commented 1 year ago

Codecov Report

Merging #279 (9666267) into 12.0 (8b44e1e) will not change coverage. The diff coverage is n/a.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more

@@           Coverage Diff           @@
##             12.0     #279   +/-   ##
=======================================
  Coverage   68.41%   68.41%           
=======================================
  Files         132      132           
  Lines        1738     1738           
  Branches      325      325           
=======================================
  Hits         1189     1189           
  Misses        515      515           
  Partials       34       34           

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

huguesdk commented 1 year ago

i’ve manually changed the cron yesterday on cie’s instance to match this change, and the overtime was correctly computed.

carmenbianca commented 1 year ago

/ocabot merge minor

github-grap-bot commented 1 year ago

Hey, thanks for contributing! Proceeding to merge this for you. Prepared branch 12.0-ocabot-merge-pr-279-by-carmenbianca-bump-minor, awaiting test results.

github-grap-bot commented 1 year ago

It looks like something changed on 12.0 in the meantime. Let me try again (no action is required from you). Prepared branch 12.0-ocabot-merge-pr-279-by-carmenbianca-bump-minor, awaiting test results.

github-grap-bot commented 1 year ago

Congratulations, your PR was merged at 99128835d8f253fa23fe6a4001d75ca6495958a1. Thanks a lot for contributing to coopiteasy. ❤️