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

[16.0][IMP] stock_picking_mass_action: Added the possibility to indicate to the method if we want other types of operation. #1490

Merged

Conversation

RodrigoBM
Copy link
Contributor

No description provided.

@RodrigoBM RodrigoBM changed the title [IMP] stock_picking_mass_action: Added the possibility to indicate to the method if we want other types of operation. [16.0][IMP] stock_picking_mass_action: Added the possibility to indicate to the method if we want other types of operation. Jan 24, 2024
@RodrigoBM RodrigoBM force-pushed the 16.0-imp-stock_picking_mass_action branch 4 times, most recently from 3240545 to 1245dc9 Compare January 25, 2024 11:48
@RodrigoBM RodrigoBM marked this pull request as ready for review January 26, 2024 09:03
@RodrigoBM RodrigoBM force-pushed the 16.0-imp-stock_picking_mass_action branch from 9c3e0a0 to 4858ecb Compare January 26, 2024 10:11
@FranciscoFactorLibre
Copy link
Contributor

I have checked and it seems correct.

I show applied flow, and tests.

Configure warehouse

  1. Active Multi-Step Routes

image

  1. Create new warehouse, outgoing configuration (2 Steps)

image

Sales flow and Stock Pikcing create

  1. Create and confirm orders in both warehouses
2024-01-29.13-31-22.mp4
  1. Add stock in both warehouses, for products
2024-01-29.13-34-43.mp4

Check Availability

  1. Filter Picking

image

  1. Check Availability without context
2024-01-29.13-40-58.mp4
  1. Check Availability with context operation type outgoing and incoming
    • Context used: model.check_assign_all(other_domain=[("picking_type_code", "in", ["outgoing", "incoming"])])
2024-01-29.13-43-47.mp4
  1. Check Availability with context operation type outgoing and internal
    • Context used: model.check_assign_all(other_domain=[("picking_type_code", "in", ["outgoing", "internal"])])
2024-01-29.13-49-54.mp4
  1. Check Availability with context operation type outgoing and internal and picking type id 2,15
    • Context used: model.check_assign_all(other_domain=[("picking_type_code", "in", ["outgoing", "internal"]) and ("picking_type_id", "in", [2,15])])
    • Context used: model.check_assign_all(other_domain=[("picking_type_code", "in", ["outgoing", "internal"]) and ("picking_type_id", "in", [15])])
2024-01-29.14-13-44.mp4

This application of the functionality is very useful to manage the availability check of several warehouses, or with multi-stage warehouses automatically.

@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). 🤖

@FranciscoFactorLibre
Copy link
Contributor

Hi @rousseldenis or @carolinafernandez-tecnativa

Could you please check this PR?

Thanks

Copy link
Contributor

Choose a reason for hiding this comment

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

LGTM :) Thanks!!

Copy link

@andreampiovesana andreampiovesana left a comment

Choose a reason for hiding this comment

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

merge?

… the method if we want other domain, state is confirmed apply by default in all cases.
@RodrigoBM RodrigoBM force-pushed the 16.0-imp-stock_picking_mass_action branch from 4858ecb to 27edd69 Compare February 26, 2024 09:32
@RodrigoBM
Copy link
Contributor Author

can @rousseldenis merge it?

@rousseldenis
Copy link
Contributor

/ocabot merge minor

@OCA-git-bot
Copy link
Contributor

What a great day to merge this nice PR. Let's do it!
Prepared branch 16.0-ocabot-merge-pr-1490-by-rousseldenis-bump-minor, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit e3b46b2 into OCA:16.0 Feb 26, 2024
9 checks passed
@OCA-git-bot
Copy link
Contributor

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

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.

7 participants