[MIG] stock_available_immediately: Migration to 11.0 #778
Conversation
(lp:c2c-addons/6.1 rev 28.1.10)
(lp:c2c-addons/6.1 rev 45.1.7)
…PL-3 with author agreement, improved help messages and pep8 (lp:c2c-addons/6.1 rev 45.1.8)
[UPG] Upgraded to version 8, fixed references to new 8.0 views and moved fields that were in product.product to product.template [fix] remove duplicate view and correct view name [UPG][FIX] added outgoing field XML, that was in the base stock field in 7.0. [UPG] stock available immediately, corrected the calculation method of immediately_usable_qty to take in accountthe sign change in outgoing_qty (from negative to positive) in version 8. [FLAKE8] [FIX] renaming of a class, comment removing, useless code. [UPD] move out from unported to 8 for update [fix] remove duplicate view and correct view name [UPG][FIX] added outgoing field XML, that was in the base stock field in 7.0. [FIX] renaming of a class, comment removing, useless code. [fix] remove duplicate view and correct view name [UPG][FIX] added outgoing field XML, that was in the base stock field in 7.0. [UPD] move out from unported to 8 for update [fix] remove duplicate view and correct view name [UPG][FIX] added outgoing field XML, that was in the base stock field in 7.0. [FIX] renaming of a class, comment removing, useless code. [UPD] move out from unported to 8 for update [fix] remove duplicate view and correct view name [UPG][FIX] added outgoing field XML, that was in the base stock field in 7.0. [FIX] renaming of a class, comment removing, useless code. [UPD] move out from unported to 8 for update [fix] remove duplicate view and correct view name [UPG][FIX] added outgoing field XML, that was in the base stock field in 7.0. [FIX] renaming of a class, comment removing, useless code. [fix] remove duplicate view and correct view name [UPG][FIX] added outgoing field XML, that was in the base stock field in 7.0. [UPD] move out from unported to 8 for update [fix] remove duplicate view and correct view name [UPG][FIX] added outgoing field XML, that was in the base stock field in 7.0. [FIX] renaming of a class, comment removing, useless code.
[UPG] Upgraded to version 8, fixed references to new 8.0 views and moved fields that were in product.product to product.template [RFR] Removing useless code and renaming
Generic module to compute the stock quantity available to promise using several implementations. stock_available_immediatly is changed to become the first optional implementation. Cherry pick of commit 0b060f6 from the v7 branch
Cherry-picked from v7 at 8add4be Conflicts: __unported__/stock_available_mrp/__openerp__.py stock_available/__openerp__.py stock_available_immediately/__openerp__.py
…uct and product.template, now takes in account variants and correctly displays value. [FLAKE8] Removing duplicate modules and moving README.rst into __unported__
calculations done in product template are fine.
This reverts commit c74aab4.
…he quant calculations done in product template are fine." This reverts commit d6dee6f.
The previous fix restored stock_available but then there was no way to exclude the incomming moves from the count. This belongs in stock_available_immediately, restoring it cleanly. This commit also takes care to respect the distinction between templates and variants, so it should fix #73 too.
|
Hey @dmsergio, thank you for your Pull Request. It looks like some users haven't signed our Contributor License Agreement, yet.
Appreciation of efforts, |
|
Hello @pedrobaeza, have I preserved correctly the history commit? |
|
Yes, thanks. Check now why Travis is red. |
|
I think it's not your fault, but |
The tests of |
|
Any news about this PR? |
|
You have to find reviewers of the PR. You can review others and ask in exchange that they review yours. |
|
@dmsergio, I think the changes made are correct, we have missed the migration of this module |
|
Please use GitHub's approval mechanism for proper visibility. |
|
This PR has the |
|
Although none of the reviewers is PSC of this repo, I think we can trust being a migration and of something very confined, so /ocabot merge |
|
This PR looks fantastic, let's merge it! |
|
Congratulations, your PR was merged at 44842f9. Thanks a lot for contributing to OCA. ❤️ |
No description provided.