[16.0][FIX] mail_tracking: bounces and others

[chienandalu]

Some improvements/fixes detected in #1367 and brought to 16.0:

• Add mail_tracking_ids to _compute_is_failed_message triggers.
• Handle bounces to avoid flagging recipients which aren't bounced.
• Add a logger.debug to research tracking issues
• Make the links in the trackings respect the breadcrumb actions.

cc @Tecnativa

[pedrobaeza]

Can you check CI?

[chienandalu]

Looks like the branch is red due to mail_composer_cc_bcc TestMailCcBcc.test_upstream_file_hash test

That module has the mail.mail._send method patched and that test is a way to check if the hash for the upstream method has changed, which has happened last week...: OCA/OCB@d492bbd

It's a good way to check if your patch is missing something... but it blocks everything else when it does...

So it would be needed to fix the patched method and set the new hash

cc @hailangvn2023 @sonhd91 @sonhd91 @nilshamerlinck @hbrunn

[nilshamerlinck]

Hello @chienandalu thanks for the heads up! created #1407 and #1408 to do that

[pedrobaeza]

Fast-tracking this bugfix:

/ocabot merge patch

[OCA-git-bot]

On my way to merge this fine PR!
Prepared branch 16.0-ocabot-merge-pr-1406-by-pedrobaeza-bump-patch, awaiting test results.

[OCA-git-bot]

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