-
-
Notifications
You must be signed in to change notification settings - Fork 649
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
[11.0][MIG] stock_picking_back2draft #550
[11.0][MIG] stock_picking_back2draft #550
Conversation
This module allows to bring cancelled pickings back to draft Just click on 'back to draft' button
v12 or v11? commit says 12 but is pointing to v11 branch. |
Why do you need this? Unlock the picking and modify whatever... |
In fact, this can lead to some inconsistencies in current flows. |
da112c9
to
d68fd4c
Compare
@pedrobaeza In some situations, people cancelled pickings (manipulation error or ....). If you cannot set it back to draft, it is impossible easily to solve the situation |
OK, if you think there's no problem on this, go ahead. |
v11? 😄
Hi @pedrobaeza I did not notice about that unlock option, is it standard?
Same as in previous versions of the module, but it helps to fix cancellation by mistake... |
Yes, the unlock option is standard. I don't remember if it's enabled by default or you need to enable it on the settings. But this option is on done pickings, not on cancelled ones. |
@aheficent What's the status of this ? |
@rousseldenis I think this is still ready to review. The objective of this module is to allow quick fixes by users when cancelling stock pickings, same as in previous versions. I think it is still valid. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Code review
d68fd4c
to
ea34d4a
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tested on runbot. Functional review 👍
This PR has the |
/ocabot merge |
This PR looks fantastic, let's merge it! |
Congratulations, your PR was merged at d30d67d. Thanks a lot for contributing to OCA. ❤️ |
…_link [12.0][MIG] stock_orderpoint_move_link
Nothing to do