search

OCA / stock-logistics-reporting

Provides Odoo Logistics reporting modules
https://odoo-community.org/psc-teams/logistics-18
GNU Affero General Public License v3.0
66 stars 232 forks source link

[16.0][ADD] stock_average_daily_sale #257

Closed rousseldenis closed 6 months ago

rousseldenis commented 1 year ago

Depends on:

I've put a comment there : https://github.com/OCA/wms/pull/496#pullrequestreview-1247246210

Because the dependency on stock_storage_type should be avoided

simahawk commented 1 year ago

/ocabot migration stock_average_daily_sale

github-actions[bot] commented 1 year ago

There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days. If you want this PR to never become stale, please ask a PSC member to apply the "no stale" label.

sbejaoui commented 1 year ago

@jbaudoux , can you update your review please

rousseldenis commented 1 year ago

@jbaudoux Could you update your review ?

OCA-git-bot commented 1 year ago

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

rousseldenis commented 1 year ago

What about adding the proposed help?

If for average_daily_sales_count, it is already there.

dreispt commented 11 months ago

/ocabot merge nobump

OCA-git-bot commented 11 months ago

This PR looks fantastic, let's merge it! Prepared branch 16.0-ocabot-merge-pr-257-by-dreispt-bump-nobump, awaiting test results.

OCA-git-bot commented 11 months ago

@dreispt your merge command was aborted due to failed check(s), which you can inspect on this commit of 16.0-ocabot-merge-pr-257-by-dreispt-bump-nobump.

After fixing the problem, you can re-issue a merge command. Please refrain from merging manually as it will most probably make the target branch red.

github-actions[bot] commented 6 months ago

There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days. If you want this PR to never become stale, please ask a PSC member to apply the "no stale" label.

jbaudoux commented 6 months ago

/ocabot merge nobump

OCA-git-bot commented 6 months ago

This PR looks fantastic, let's merge it! Prepared branch 16.0-ocabot-merge-pr-257-by-jbaudoux-bump-nobump, awaiting test results.

OCA-git-bot commented 6 months ago

Congratulations, your PR was merged at ef5b10867d57462b2aa608691ae533c76180fac2. Thanks a lot for contributing to OCA. ❤️