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

[13.0][IMP] mrp_multi_level #470

Merged
merged 1 commit into from
Mar 11, 2020
Merged

[13.0][IMP] mrp_multi_level #470

merged 1 commit into from
Mar 11, 2020

Conversation

HviorForgeFlow
Copy link
Member

@HviorForgeFlow HviorForgeFlow commented Mar 2, 2020 •
edited
Loading

[IMP] Minor changes:

  • Planned Order release and due date become required.
  • Add button to Product MRP Area to update MOQ from Supplier Info.
  • Link Manufacturing Orders with Planned Orders.
  • Allow Mrp Inventory Procure Wizard to be used from other models.
  • Make MRP Inventory creation more extensible.
  • Main Supplier computation (v13 requires explicit False definitions).

CC @ForgeFlow @LoisRForgeFlow

@OCA-git-bot
Copy link
Contributor

Hi @LoisRForgeFlow, @JordiBForgeFlow,
some modules you are maintaining are being modified, check this out!

@HviorForgeFlow HviorForgeFlow force-pushed the 13.0-imp-mrp_multi_level-planned_order_dates_required branch from 7dc9558 to ae98e4c Compare March 3, 2020 12:30
@HviorForgeFlow HviorForgeFlow changed the title [13.0][IMP] mrp_multi_level: Planned Order required dates [13.0][IMP] mrp_multi_level Mar 3, 2020
@HviorForgeFlow HviorForgeFlow force-pushed the 13.0-imp-mrp_multi_level-planned_order_dates_required branch 13 times, most recently from 31e1fed to 2bdcb77 Compare March 10, 2020 10:00
LoisRForgeFlow
LoisRForgeFlow reviewed Mar 10, 2020
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.

Looks great, just some small comments:

mrp_multi_level/readme/HISTORY.rst Outdated Show resolved Hide resolved
mrp_multi_level/models/product_mrp_area.py Outdated Show resolved Hide resolved
mrp_multi_level/views/product_mrp_area_views.xml Outdated Show resolved Hide resolved
mrp_multi_level/wizards/mrp_inventory_procure.py Outdated Show resolved Hide resolved
@HviorForgeFlow HviorForgeFlow force-pushed the 13.0-imp-mrp_multi_level-planned_order_dates_required branch 7 times, most recently from eb260ae to 5f3ca42 Compare March 11, 2020 08:19
@HviorForgeFlow
[IMP] mrp_multi_level:
eab1941 
* Planned Order release and due date become required.
* Add button to Product MRP Area to update MOQ from Supplier Info.
* Link Manufacturing Orders with Planned Orders.
* Allow Mrp Inventory Procure Wizard to be used from other models.
* Make MRP Inventory creation more extensible.
* Main Supplier computation (v13 requires explicit False definitions).
LoisRForgeFlow
LoisRForgeFlow approved these changes Mar 11, 2020
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.

LGTM 👍

@HviorForgeFlow HviorForgeFlow force-pushed the 13.0-imp-mrp_multi_level-planned_order_dates_required branch from 5f3ca42 to eab1941 Compare March 11, 2020 08:21
@LoisRForgeFlow
Copy link
Contributor

/ocabot merge minor

@OCA-git-bot
Copy link
Contributor

Hey, thanks for contributing! Proceeding to merge this for you.
Prepared branch 13.0-ocabot-merge-pr-470-by-LoisRForgeFlow-bump-minor, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 2e228bb into OCA:13.0 Mar 11, 2020
@OCA-git-bot
Copy link
Contributor

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

@LoisRForgeFlow LoisRForgeFlow deleted the 13.0-imp-mrp_multi_level-planned_order_dates_required branch March 11, 2020 09:35
CarlosRoca13 pushed a commit to Tecnativa/manufacture that referenced this pull request Feb 15, 2021
@OCA-git-bot
Merge PR OCA#470 into 13.0
44bf6d8 
Signed-off-by LoisRForgeFlow
CarlosRoca13 pushed a commit to Tecnativa/manufacture that referenced this pull request Feb 22, 2021
@OCA-git-bot
Merge PR OCA#470 into 13.0
ae14003 
Signed-off-by LoisRForgeFlow
CarlosRoca13 pushed a commit to Tecnativa/manufacture that referenced this pull request Mar 2, 2021
@OCA-git-bot
Merge PR OCA#470 into 13.0
e77f7ed 
Signed-off-by LoisRForgeFlow
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@LoisRForgeFlow LoisRForgeFlow LoisRForgeFlow approved these changes

Assignees
No one assigned
Labels
merged 🎉
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@HviorForgeFlow @OCA-git-bot @LoisRForgeFlow