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

[12.0][MIG] stock_orderpoint_manual_procurement #557

Merged
merged 17 commits into from Apr 4, 2019
Merged

[12.0][MIG] stock_orderpoint_manual_procurement #557

merged 17 commits into from Apr 4, 2019

Conversation

kittiu
Copy link
Member

@kittiu kittiu commented Mar 16, 2019

Module migrated to v12

JordiBForgeFlow and others added 16 commits March 16, 2019 12:05
@JordiBForgeFlow @kittiu
stock_orderpoint_manual_procurement (OCA#210)
4f47e12 
[ADD] stock_orderpoint_procure_uom
@LoisRForgeFlow @kittiu
stock_orderpoint_manual_procurement: Migrated to 9.0
150b6ee
@LoisRForgeFlow @kittiu
[10.0][MIG] stock_orderpoint_manual_procurement
3e1fb5f
@LoisRForgeFlow @kittiu
[10.0][REW] remove dependency with stock_orderpoint_uom
928954f
@kittiu
[MIG] stock_orderpoint_manual_procurement: Migration to 11.0
43117d3
@kittiu
[IMP] wizard button standard & QTY readonly if no access
43a8de4
@kittiu
[FIX] orderpoint quantity
191fcad
@kittiu
[IMP] set UoM values from wizard line UoM
959e5f8
@LoisRForgeFlow @kittiu
[11.0][FIX] stock_orderpoint_manual_procurement: readd date_planned t…
0263514 
…o wizard.
@LoisRForgeFlow @kittiu
[11.0][IMP] stock_orderpoint_manual_procurement: add test case
6d5efe4
@grindtildeath @kittiu
[Fix] Convert date_planned string to datetime string
5805e4b
@oca-travis @kittiu
[UPD] Update stock_orderpoint_manual_procurement.pot
3fddab7
@oca-travis @kittiu
[UPD] Update stock_orderpoint_manual_procurement.pot
4848eda
@AdriaGForgeFlow @kittiu
[11.0][IMP] Improve module stock_orderpoint_manual_procurement
9c0f906
@LoisRForgeFlow @kittiu
[11.0][FIX] stock_orderpoint_manual_procurement: pass requested_uid i…
be217cb 
…n context to be able to have it through a chain of procuremets.
@oca-travis @kittiu
[UPD] Update stock_orderpoint_manual_procurement.pot
3132d09
@kittiu kittiu mentioned this pull request Mar 16, 2019
Closed
@kittiu kittiu changed the title [12.0][MIG] stock_orderpoint_purchase_link [12.0][MIG] stock_orderpoint_manual_procurement Mar 16, 2019
@OCA-git-bot OCA-git-bot mentioned this pull request Mar 18, 2019
Closed
23 tasks
@rousseldenis rousseldenis added this to the 12.0 milestone Mar 18, 2019
@OCA-git-bot OCA-git-bot mentioned this pull request Mar 18, 2019
Closed
54 tasks
@LoisRForgeFlow
Copy link
Contributor

@kittiu Please check warnings in runbot logs

@kittiu kittiu force-pushed the 12.0-mig-stock_orderpoint_manual_procurement branch from fd71901 to eefe302 Compare March 29, 2019 04:41
LoisRForgeFlow
LoisRForgeFlow approved these changes Apr 1, 2019
View reviewed changes
Copy link
Contributor

@LoisRForgeFlow LoisRForgeFlow left a comment

Choose a reason for hiding this comment

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

Tested locally, LGTM 👍

bodedra
bodedra approved these changes Apr 2, 2019
View reviewed changes
Copy link
Member

@bodedra bodedra left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@bodedra
Copy link
Member

bodedra commented Apr 2, 2019

@kittiu Appreciate your work. IMO we can avoid cosmetic changes on next module migration. 😉 That's not really important on migration. Reference: eefe302#diff-caa1ed47e650d4d52dc2c98f85bee472R99

@OCA-git-bot
Copy link
Contributor

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
rousseldenis requested changes Apr 3, 2019
View reviewed changes
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.

@kittiu Could you amend commit message ?

@kittiu kittiu force-pushed the 12.0-mig-stock_orderpoint_manual_procurement branch from eefe302 to 495f2ba Compare April 3, 2019 08:07
@kittiu
Copy link
Member Author

kittiu commented Apr 3, 2019

@kittiu Could you amend commit message ?

Done

@kittiu kittiu mentioned this pull request Apr 3, 2019
Merged
2 tasks
@OCA-git-bot
Copy link
Contributor

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 rousseldenis merged commit 70f7bf2 into OCA:12.0 Apr 4, 2019
LRovira-PlanetaTIC pushed a commit to PlanetaTIC/stock-logistics-warehouse that referenced this pull request May 15, 2020
@OCA-git-bot
Merge PR OCA#557 into 10.0
0f6d2cb 
Signed-off-by lreficent
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@bodedra bodedra bodedra approved these changes

@rousseldenis rousseldenis rousseldenis approved these changes

@LoisRForgeFlow LoisRForgeFlow LoisRForgeFlow approved these changes

@JordiBForgeFlow JordiBForgeFlow Awaiting requested review from JordiBForgeFlow

Assignees
No one assigned
Labels
approved ready to merge
Projects
None yet
Milestone
12.0
Development

Successfully merging this pull request may close these issues.

None yet
9 participants
@kittiu @LoisRForgeFlow @bodedra @OCA-git-bot @rousseldenis @JordiBForgeFlow @grindtildeath @oca-travis @AdriaGForgeFlow