social icon indicating copy to clipboard operation
social copied to clipboard

[15.0][IMP] mail_composer_cc_bcc: filter out partner with no email address and set "" for partner with no name

Open sonhd91 opened this issue 1 year ago • 2 comments

  • Normally a wizard opens on Odoo client side to prompt user to define an email address for partners that don't have one
  • But it's possible to bypass it, so we explicitly filter out partners with no email address
  • We also return "" instead of "False" for partner with no name, and odoo's tools.formataddr will just use the partner's email address

sonhd91 avatar Feb 06 '24 09:02 sonhd91

Hi @hailangvn2023, some modules you are maintaining are being modified, check this out!

OCA-git-bot avatar Feb 06 '24 09:02 OCA-git-bot

There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days. If you want this PR to never become stale, please ask a PSC member to apply the "no stale" label.

github-actions[bot] avatar Jul 28 '24 12:07 github-actions[bot]

Hello, can this PR be reopened and merged? Unless that I have opened a new one to be able to do a followup if requested.

leemannd avatar Nov 05 '24 09:11 leemannd

Hi @hailangvn2023, some modules you are maintaining are being modified, check this out!

OCA-git-bot avatar Nov 05 '24 10:11 OCA-git-bot

/ocabot merge minor

no idea what's going on with runboat, and I'm overriding @hailangvn2023 as this handle seems inactive. The module should get a new maintainer then or be orphaned

hbrunn avatar Nov 05 '24 13:11 hbrunn

What a great day to merge this nice PR. Let's do it! Prepared branch 15.0-ocabot-merge-pr-1293-by-hbrunn-bump-minor, awaiting test results.

OCA-git-bot avatar Nov 05 '24 13:11 OCA-git-bot

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

OCA-git-bot avatar Nov 05 '24 13:11 OCA-git-bot