[17.0][MIG] fetchmail_notify_error_to_sender, fetchmail_notify_error_to_sender_test: Migration to 17.0
fetchmail_notify_error_to_sender: Migration from 15.0 fetchmail_notify_error_to_sender_test: Migration from 14.0
Fixes the bug found here
Modified the tests to include all the raise conditions.
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). 🤖
Hi @PieterPaulussen, sorry for the late reply and thx for the review, I agree with all your points.
I am not sure if they can all go in the same commit as the migration, or if I need to use a different commit and syntax for the new commit message, since they are improvements not directly related with the migration but with the module.
I think it's perfectly fine to address @PieterPaulussen's comments in a new commit in the same PR
Thanks for the updates @BT-cjimeno
/ocabot merge nobump
This PR looks fantastic, let's merge it! Prepared branch 17.0-ocabot-merge-pr-2982-by-hbrunn-bump-nobump, awaiting test results.
Congratulations, your PR was merged at 39147d117f71b548517b47f9a9385763c495cc60. Thanks a lot for contributing to OCA. ❤️