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

new method to validate moves in a separate transfer #72

Merged

Conversation

sebalix
Copy link
Member

@sebalix sebalix commented Sep 16, 2020

No description provided.

@sebalix sebalix force-pushed the 13.0-shopfloor-action_done_in_picking branch from b81b880 to 65409b0 Compare September 16, 2020 14:16
@sebalix sebalix changed the title [WIP] new method to validate moves in a separate transfer new method to validate moves in a separate transfer Sep 16, 2020
@sebalix sebalix force-pushed the 13.0-shopfloor-action_done_in_picking branch 4 times, most recently from 3397ddb to 4aafbf2 Compare September 16, 2020 15:26
@sebalix sebalix force-pushed the 13.0-shopfloor-action_done_in_picking branch from ba3939b to 6100ed0 Compare September 17, 2020 07:52
@sebalix
Copy link
Member Author

sebalix commented Sep 17, 2020

Rebased to get a green build

@codecov-commenter
Copy link

codecov-commenter commented Sep 17, 2020

Codecov Report

Merging #72 into 13.0-shopfloor will increase coverage by 0.01%.
The diff coverage is 94.73%.

Impacted file tree graph

@@                Coverage Diff                 @@
##           13.0-shopfloor      #72      +/-   ##
==================================================
+ Coverage           91.95%   91.96%   +0.01%     
==================================================
  Files                  54       54              
  Lines                3866     3884      +18     
==================================================
+ Hits                 3555     3572      +17     
- Misses                311      312       +1     
Impacted Files Coverage Δ
shopfloor/models/stock_move.py 97.67% <94.73%> (-2.33%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 18624c8...7e93e5a. Read the comment docs.

@sebalix
Copy link
Member Author

sebalix commented Sep 17, 2020

test_stock_split.py has never been imported, that's why codecov was not happy

@guewen guewen merged commit dbec76e into camptocamp:13.0-shopfloor Sep 17, 2020
simahawk pushed a commit to simahawk/wms that referenced this pull request Oct 12, 2020
Calling _action_done() directly on the stock.move while leaving the
stock.picking "assigned" with the other moves already required various
workarounds as it is normally not an expected flow in Odoo.

This method is to be used in services instead of calling _action_done() on
the moves. If a move to set to done contains move lines that should not
be validated, the line to keep should be extracted beforehand with
`move_id.split_other_move_lines`.
guewen pushed a commit that referenced this pull request Nov 16, 2020
Signed-off-by simahawk
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants