Skip to content

Conversation

@Abridbus
Copy link
Contributor

Results of seeing a lot of wrong and missing configuration on consultancy project for Outlook accounts.

The Multi outgoing mail server is harder to maintain than having a unique mail server with an Odoo db correctly configured.

Insisting on the mail.default.from (ICP) and the From Filtering parameters that NEEDS TO BE set up.

task-3512379

@Abridbus Abridbus requested a review from tiku-odoo September 25, 2023 13:55
@robodoo
Copy link
Collaborator

robodoo commented Sep 25, 2023

@StraubCreative
Copy link
Contributor

@tiku-odoo can you verify the content and then tag @samueljlieber for technical review, please?

Copy link
Contributor

@jqu-odoo jqu-odoo left a comment

Choose a reason for hiding this comment

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

Here is my own summary of the issue and perhaps the basis for a different wording.

Configuration with a single outgoing mail server

You use a generic email address, e.g. notifications@example.com or odoo@example.com, connected as an outgoing mail server.

This address must be set as from_filter on the server.

This address must also match the mail.default.from@mail.catchall.domain key combination in the system parameters.

Configuration sample :

Multiple user configuration

In addition to the generic server, you can use an outgoing mail server for each user. The email address of each user must be set as from_filter on the server.

Sample configuration :

Bottom line: we see a lot of misconfigurations because mail.default.from is edited to use a full email address, which does not belong to mail.catchall.domain. mail.catchall.domain should be edited to match the From filter.

@tiku-odoo tiku-odoo force-pushed the 15.0-outlook_unique_mailbox_config-BVE branch from 7093b12 to 2f869b8 Compare September 27, 2023 14:58
@tiku-odoo tiku-odoo requested a review from jqu-odoo September 27, 2023 14:59
@tiku-odoo tiku-odoo self-assigned this Sep 27, 2023
@tiku-odoo
Copy link
Contributor

@Abridbus @jqu-odoo

I've updated this PR to include the info you mentioned in your previous comments/edits. Can you review and stamp your approval before we move this over for a technical (RST) Markdown review?

Thanks in advance for your help on this!

👍

CC: @StraubCreative

@tiku-odoo
Copy link
Contributor

@jqu-odoo -- Hi Jonathan, when you have a moment, can you review this doc for technical accuracy? Thanks in advance for your help on this. Great suggestions so far!

Thanks, Tim 👍

@tiku-odoo tiku-odoo force-pushed the 15.0-outlook_unique_mailbox_config-BVE branch from 2f869b8 to 59c8951 Compare September 29, 2023 20:19
@tiku-odoo tiku-odoo force-pushed the 15.0-outlook_unique_mailbox_config-BVE branch from 59c8951 to d8bb4ad Compare October 2, 2023 14:35
@tiku-odoo tiku-odoo requested review from Felicious and samueljlieber and removed request for Felicious October 2, 2023 14:40
@tiku-odoo tiku-odoo force-pushed the 15.0-outlook_unique_mailbox_config-BVE branch from d8bb4ad to 6326754 Compare October 2, 2023 14:51
@tiku-odoo
Copy link
Contributor

@samueljlieber This doc is ready for your technical review.

@tiku-odoo tiku-odoo force-pushed the 15.0-outlook_unique_mailbox_config-BVE branch from 6326754 to 34708c2 Compare October 3, 2023 17:52
Copy link
Contributor

@samueljlieber samueljlieber left a comment

Choose a reason for hiding this comment

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

Hi @Abridbus and @tiku-odoo, this PR is looking good! I have just a couple technical changes and some suggestions, listed below. Please let me know if you have any questions and your thoughts on the link labels.

Thank you for the continued hard work!!

Results of seeing a lot of wrong and missing configuration
on consultancy project for Outlook accounts.

The Multi outgoing mail server is harder to maintain than having
a unique mail server with an Odoo db correctly configured.

Insisting on the mail.default.from (ICP) and the From Filtering
parameters that NEEDS TO BE set up.

task-3512379

[FIX] Maintain: Azure_oauth: configuration mail server Outlook

Co-author-by: tiku-odoo <tiku@odoo.com>
Co-author-by: jqu-odoo <jqu@odoo.com>

Results of seeing a lot of wrong and missing configuration
on consultancy project for Outlook accounts.

The Multi outgoing mail server is harder to maintain than having
a unique mail server with an Odoo db correctly configured.

Insisting on the mail.default.from (ICP) and the From Filtering
parameters that NEEDS TO BE set up.

task-3512379
@tiku-odoo tiku-odoo force-pushed the 15.0-outlook_unique_mailbox_config-BVE branch from 34708c2 to f721908 Compare October 3, 2023 20:58
@tiku-odoo
Copy link
Contributor

@samueljlieber I've made the changes you requested. Thanks for your review of this doc.

Ready to move to ZST, I believe.

Thanks 👍

Copy link
Contributor

@samueljlieber samueljlieber left a comment

Choose a reason for hiding this comment

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

Looks good @tiku-odoo thanks! 🙂
On to you @StraubCreative

@StraubCreative
Copy link
Contributor

LGTM 👍
Thanks @Abridbus and team!

@robodoo r+

robodoo pushed a commit that referenced this pull request Oct 5, 2023
Results of seeing a lot of wrong and missing configuration
on consultancy project for Outlook accounts.

The Multi outgoing mail server is harder to maintain than having
a unique mail server with an Odoo db correctly configured.

Insisting on the mail.default.from (ICP) and the From Filtering
parameters that NEEDS TO BE set up.

task-3512379

[FIX] Maintain: Azure_oauth: configuration mail server Outlook

Co-author-by: tiku-odoo <tiku@odoo.com>
Co-author-by: jqu-odoo <jqu@odoo.com>

Results of seeing a lot of wrong and missing configuration
on consultancy project for Outlook accounts.

The Multi outgoing mail server is harder to maintain than having
a unique mail server with an Odoo db correctly configured.

Insisting on the mail.default.from (ICP) and the From Filtering
parameters that NEEDS TO BE set up.

task-3512379

closes #5909

Signed-off-by: Zachary Straub (zst) <zst@odoo.com>
@robodoo robodoo temporarily deployed to merge October 5, 2023 18:58 Inactive
@robodoo robodoo closed this Oct 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants