OCA / account-analytic

Odoo Account Analytic Related Addons
GNU Affero General Public License v3.0
101 stars 358 forks source link

[17.0][MIG] mrp_stock_analytic #644

Open aisopuro opened 4 months ago

aisopuro commented 4 months ago

Depends on

Notes

github-actions[bot] commented 2 weeks 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.

aisopuro commented 2 weeks ago

Rebased on top of 17.0. @StefanRijnhart I think this is still active, are you able to mark it not stale?

StefanRijnhart commented 2 weeks ago

@aisopuro We should be fine if we keep on being active on it! Can you check the failing tests?

aisopuro commented 2 weeks ago

I took a look, but the trace suggests the failure is in stock_analytic, which is green in 17.0. So its probably some kind of non-trivial interaction, so I unfortunately probably won't manage to get to it anytime soon.

StefanRijnhart commented 2 weeks ago

@aisopuro then this one will become stale again I'm afraid

aisopuro commented 1 day ago

@StefanRijnhart I've fixed comments and rebased, this PR is good to go again.

StefanRijnhart commented 1 day ago

@aisopuro Great work on the tests, thanks!