OCA
[17.0][MIG] stock_helper
@LoisRForgeFlow @jbaudoux An improvement I requested was to refactor the module name in order to avoid putting a lot of things inside it without context.
stock_location_sublocation_of / stock_location_from_route /... is better
or at least stock_helper_location_sublocation / stock_helper_location_from_route if we want to keep the 'helper' term
@LoisRForgeFlow @jbaudoux An improvement I requested was to refactor the module name in order to avoid putting a lot of things inside it without context.
stock_location_sublocation_of / stock_location_from_route /... is better
or at least stock_helper_location_sublocation / stock_helper_location_from_route if we want to keep the 'helper' term
Right now it is only 2 helpers and none of them makes or generates any problem if not used. I don't know if such atomization would provide real value in this case.
My 2 cents
On my way to merge this fine PR! Prepared branch 17.0-ocabot-merge-pr-1959-by-LoisRForgeFlow-bump-nobump, awaiting test results.
Congratulations, your PR was merged at 8f179ba58e74fc2e8124b23c851ad67ac4035845. Thanks a lot for contributing to OCA. ❤️