Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Revert "[IMP] stock_available_mrp: Easier and faster BoM finding" #784

Merged
merged 1 commit into from
Dec 4, 2019

Conversation

Cedric-Pigeon
Copy link

@Cedric-Pigeon Cedric-Pigeon commented Dec 3, 2019

This reverts commit 4be9edb related to #611
The related PR has been merged too quickly as it depends from an Odoo Pr not merged.
As this it decreases performances.

@pedrobaeza pedrobaeza added this to the 10.0 milestone Dec 3, 2019
Copy link
Sponsor Contributor

@rousseldenis rousseldenis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Indeed

@pedrobaeza
Copy link
Member

/ocabot merge patch

@OCA-git-bot
Copy link
Contributor

This PR looks fantastic, let's merge it!
Prepared branch 10.0-ocabot-merge-pr-784-by-pedrobaeza-bump-patch, awaiting test results.

OCA-git-bot added a commit that referenced this pull request Dec 4, 2019
Signed-off-by pedrobaeza
@yajo
Copy link
Member

yajo commented Dec 4, 2019

I guess I'll have to reopen #611 after this, in WIP this time until odoo/odoo#33967 is merged.

@pedrobaeza
Copy link
Member

Yes, but I don't think this will be merged in Odoo. We have escaped from 10.0 any way, but not everyone are so lucky (or it's in its way of working).

@OCA-git-bot OCA-git-bot merged commit 96a1733 into OCA:10.0 Dec 4, 2019
@OCA-git-bot
Copy link
Contributor

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

@yajo
Copy link
Member

yajo commented Dec 5, 2019

But it affects all versions.

@pedrobaeza
Copy link
Member

Then do it for a supported version, not for 10.0 (I know it was supported when you did it, but now that can be a problem as it's not supported anymore).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants