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.

Sign up for GitHub

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

Jump to bottom

[11.0][MIG] stock reserve sale #886

Closed
wants to merge 40 commits into from
Closed

[11.0][MIG] stock reserve sale #886

wants to merge 40 commits into from

Conversation

RodrigoBM
Copy link

Standard migration

This PR depend to approval this other #883

Thanks

guewen and others added 30 commits April 16, 2020 16:26
@guewen @RodrigoBM
[ADD] started stock_reserve_sale to reserve lines of quotations
0d69655
@guewen @RodrigoBM
[IMP] continued the stock_reserve_sale module
5c85a40
@guewen @RodrigoBM
[ADD] release and update methods
6929da2
@guewen @RodrigoBM
[ADD] started to handle the modification of sale order lines when a r…
3c1588d 
…eservation exists.

product and uom ask to release the reservation before changing,
price_unit is copied to the reservation move
a change of quantity will warn the user to update the reservation manually (the user can choose a different quantity to reserve)
@guewen @RodrigoBM
[IMP] warn when the quantity of a line with a reservation is changed …
5489e27 
…that the qty of the reservation will be changed
@guewen @RodrigoBM
[CHG] should not be able to change the type (mto, mts) of a sales lin…
10d2bd3 
…e with a stock reservation
@guewen @RodrigoBM
[CHG] release the reservations on the confirmation of sales orders
f6d2a1f
@guewen @RodrigoBM
[FIX] wrong field name
c207091
@guewen @RodrigoBM
[CHG] reverse cardinality: many stock.reservation have 1 sale.order.l…
8d79d51 
…ine. in fact usually it would be 1-1 but make more sense to have the relation on stock.reservation
@guewen @RodrigoBM
[IMP] cancelling a sales order cancels the stock reservations
c228ca3
@guewen @RodrigoBM
[ADD] test for reservation of a sales order
eb008b3
@guewen @RodrigoBM
[ADD] test for reservation of sales order lines
3fe9fd2
@guewen @RodrigoBM
[IMP] description in __openerp__.py
0553231
@guewen @RodrigoBM
[CHG] hide the stock reservation on the line's form
4ea319f
@guewen @RodrigoBM
[FIX] in form view of sales line, remove the update button (automatic…
fdcf4e6 
…) and remove the cancel button (already displayed on the header)
@gurneyalex @RodrigoBM
mark unported module as uninstallable
d2698fd
@yvaucher @RodrigoBM
reactivate module stock_reserve_sale
e294f38
@yvaucher @RodrigoBM
port models and methods to api 8.0
646491e
@yvaucher @RodrigoBM
change product_qty to product_uom_qty
dd4207f
@yvaucher @RodrigoBM
remove procure_method from sale.order.lines
49b0c56
@yvaucher @RodrigoBM
set test product 12 to MTO
d407c11
@yvaucher @RodrigoBM
fix unit of mesure on sale order in tests
2286d27
@yvaucher @RodrigoBM
fix yaml test by writing route_ids using python instead of record
1c681b4
@yvaucher @RodrigoBM
fix name formating. str.format is recommended for python 3 not for 2.…
5bcd582 
…6. setting base string as unicode didn't solve it
@lepistone @RodrigoBM
lint
4dcb8b3
@lepistone @RodrigoBM
assert -> ensure_one
318fcb9
@lepistone @RodrigoBM
api.one -> api.multi + ensure_one
f059e2c
@gurneyalex @RodrigoBM
fix pep8 W503
14d543d
@gurneyalex @RodrigoBM
Add OCA as author of OCA addons
57a55e9 
In order to get visibility on https://www.odoo.com/apps the OCA board has
decided to add the OCA as author of all the addons maintained as part of the
association.
@lepistone @RodrigoBM
make stock_reserve_sale owner_aware
e4f1685
@RodrigoBM RodrigoBM mentioned this pull request Apr 17, 2020
Closed
46 tasks
@RodrigoBM RodrigoBM changed the title 11.0 mig stock reserve sale [11.0][MIG] stock reserve sale Apr 17, 2020
@rousseldenis rousseldenis added this to the 11.0 milestone Apr 24, 2020
@rousseldenis
Copy link
Sponsor Contributor

Have you seen that some other branches have this (maybe you are missing some commits) ?

@RodrigoBM
Copy link
Author

Hi @rousseldenis,

I do not understand your comment. Can you explain me with more details?

Thanks

@rousseldenis
Copy link
Sponsor Contributor

See https://github.com/OCA/stock-logistics-warehouse/commits/9.0/stock_reserve_sale for example.

There are some commits missing here

@RodrigoBM
Copy link
Author

it is possible, I have migrated from v10, I will review the commits from v9

https://github.com/OCA/stock-logistics-warehouse/commits/10.0/stock_reserve_sale

@github-actions
Copy link

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.

@github-actions github-actions bot added the stale PR/Issue without recent activity, it'll be soon closed automatically. label Apr 24, 2022
@github-actions github-actions bot closed this May 29, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
migration stale PR/Issue without recent activity, it'll be soon closed automatically.
Projects
None yet
Milestone
11.0
Development

Successfully merging this pull request may close these issues.

None yet
8 participants
@RodrigoBM @rousseldenis @guewen @gurneyalex @yvaucher @lepistone @mlaitinen @amarcosg