Closed victorvermot closed 3 months ago
Hi @sebalix, @jbaudoux, @mt-software-de, some modules you are maintaining are being modified, check this out!
ping @lmignon @rousseldenis
maybe we need to put singular instead of plural ?
ping @lmignon @rousseldenis
maybe we need to put singular instead of plural ?
Where ? If in buttons, no.
The forced context {"search_default_release_ready": 1}
in action_picking_all
is not very convenient as you only see by default the release ready deliveries.
I would drop it.
I removed the context @jbaudoux but it makes one test fail and I am not sure how to fix it
@victorvermot You need to adapt the test to remove what you removed from the context. See stock_release_channel.tests.test_channel_action: FAIL: TestChannelAction.test_action_all
ping @victorvermot
@victorvermot Thanks. Can you squash commits ?
@victorvermot Can you squash commits ?
@jbaudoux Done
/ocabot merge minor
This PR has the approved
label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖
What a great day to merge this nice PR. Let's do it! Prepared branch 16.0-ocabot-merge-pr-883-by-sebalix-bump-minor, awaiting test results.
It looks like something changed on 16.0
in the meantime.
Let me try again (no action is required from you).
Prepared branch 16.0-ocabot-merge-pr-883-by-sebalix-bump-minor, awaiting test results.
Congratulations, your PR was merged at 807ccf09d5e412750fa5e600e09bc5b0e2654659. Thanks a lot for contributing to OCA. ❤️
Result: