[18.0][ADD] fieldservice_sale_stock_route_backorder - #1544
Conversation
3523b03 to
c3e570a
Compare
…service_sale_stock_route_backorder
c3e570a to
864afd4
Compare
peluko00
left a comment
There was a problem hiding this comment.
LGTM! Code review and tested in runboat
|
/ocabot merge nobump |
|
This PR looks fantastic, let's merge it! |
|
@max3903 your merge command was aborted due to failed check(s), which you can inspect on this commit of 18.0-ocabot-merge-pr-1544-by-max3903-bump-nobump. After fixing the problem, you can re-issue a merge command. Please refrain from merging manually as it will most probably make the target branch red. |
| **For this module to work as intended, you need to enable the Auto | ||
| Validate FSM Pickings setting in Field Service \> Configuration \> | ||
| Settings.** |
There was a problem hiding this comment.
This feature belongs to fieldservice_stock: f75f765
I think it would make sense to enable fieldservice_stock.auto_validate_pickings in a post_init_hook, since this module depends on that feature being enabled. This would avoid requiring a manual configuration step after installation and ensure the module behaves correctly out of the box.
cc https://github.com/APSL
@miquelalzanillas @ppyczko @mpascuall @peluko00 @javierobcn @BernatObrador please review