Skip to content

[18.0][mail_cleanup]FIX : respect method signature - #3333

Merged
OCA-git-bot merged 1 commit into
OCA:18.0from
camptocamp:fix-mail_cleanup
Sep 1, 2025
Merged

OCA-git-bot merged 1 commit into
OCA:18.0from
camptocamp:fix-mail_cleanup

Conversation

@leemannd

@leemannd leemannd commented Jul 21, 2025

Copy link
Copy Markdown
Contributor

TypeError: FetchmailServer.fetch_mail() got an unexpected keyword argument 'raise_exception'

image-20250711-084847

@leemannd leemannd changed the title FIX mail_cleanup: respect method signature [18.0][mail_cleanup]FIX : respect method signature Jul 21, 2025

@Ricardoalso Ricardoalso left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@leemannd

Copy link
Copy Markdown
Contributor Author

@Ricardoalso Thanks fixed

@gurneyalex

Copy link
Copy Markdown
Member

/ocabot merge patch

@OCA-git-bot

Copy link
Copy Markdown
Contributor

Hey, thanks for contributing! Proceeding to merge this for you.
Prepared branch 18.0-ocabot-merge-pr-3333-by-gurneyalex-bump-patch, awaiting test results.

OCA-git-bot added a commit that referenced this pull request Aug 12, 2025
Signed-off-by gurneyalex
@OCA-git-bot

Copy link
Copy Markdown
Contributor

@gurneyalex your merge command was aborted due to failed check(s), which you can inspect on this commit of 18.0-ocabot-merge-pr-3333-by-gurneyalex-bump-patch.

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.

@StefanRijnhart StefanRijnhart left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! The test issue causing the merge to fail is fixed in #3343

@leemannd

leemannd commented Sep 1, 2025

Copy link
Copy Markdown
Contributor Author

Hello @StefanRijnhart , your other PR fixing the merge is merged. Do you have any time to merge this one?

@StefanRijnhart

Copy link
Copy Markdown
Member

/ocabot merge patch

@OCA-git-bot

Copy link
Copy Markdown
Contributor

What a great day to merge this nice PR. Let's do it!
Prepared branch 18.0-ocabot-merge-pr-3333-by-StefanRijnhart-bump-patch, awaiting test results.

@OCA-git-bot
OCA-git-bot merged commit 5ad5d1a into OCA:18.0 Sep 1, 2025
1 of 2 checks passed
@OCA-git-bot

Copy link
Copy Markdown
Contributor

Congratulations, your PR was merged at 22ed5dc. Thanks a lot for contributing to OCA. ❤️

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants