Michael Tietz
Michael Tietz
> > done > > 🤔 It seems to still be failing, I think just changing the test itself in that module to compare vs strings is OK, like you...
> Why can't we simply revert this #720 ? No it is fine to not copy the value, because move_dest_ids/move_orig_ids will also not copied from a move. In a backorder...
> @mt-software-de I should have added a unittest for my fix #720. If you do a partial pack, then you don't want to keep `delivery_notification_sent` on the pack backorder. So,...
> @jbaudoux your merge command was aborted due to failed check(s), which you can inspect on [this commit of 14.0-ocabot-merge-pr-767-by-jbaudoux-bump-patch](https://github.com/OCA/delivery-carrier/commits/38f72b599beef9a4067c8d96757e5beb488b5d76). > > After fixing the problem, you can re-issue a...
> @mt-software-de What's the difference with stock_reserve_sale ? > > https://github.com/OCA/stock-logistics-warehouse/pull/1788 Please read the comment of @jbaudoux https://github.com/OCA/stock-logistics-warehouse/pull/1696#issuecom This explains it very well.
https://github.com/OCA/wms/pull/769 is needed for this PR.
This one is now ready to merge, i removed the dependency to stock_helper and included the very specific logic to stop the proc run, within the module itself.
> @mt-software-de @jbaudoux What's ths status of this ? Sorry i was not working on this. I still need to check why the test are failing. On my dev env...