Skip to content

[IMP] General: Email Infinite Loops #14305

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 1 commit into from

Conversation

jero-odoo
Copy link
Contributor

@jero-odoo jero-odoo commented Aug 14, 2025

Combining multiple docs requests-
Docs task: https://www.odoo.com/mail/view?model=project.task&res_id=4989601&access_token=6db493fc-65b2-43dc-81d3-36d5510761db

Removed note regarding infinite mail loops at request from Support. 

"This section mentioned that system parameters mail.gateway.loop.minutes and mail.gateway.loop.threshold are used to prevent abuse of record-creating aliases (from Settings -> Technical Aliases) but it also mentioned that "These parameters are only used to prevent the creation of new records. They do not prevent replies from being added to the chatter." I checked the codes and notes on Github and this doesn't seem to be the case."

Docs task: https://www.odoo.com/mail/view?model=project.task&res_id=4932088&access_token=39db38ab-30a0-47fa-a76c-4a46506e255e

Added warning about personal emails

@jero-odoo jero-odoo self-assigned this Aug 14, 2025
@robodoo
Copy link
Collaborator

robodoo commented Aug 14, 2025

Pull request status dashboard

@C3POdoo C3POdoo requested a review from a team August 14, 2025 15:40
@jero-odoo jero-odoo requested review from Felicious and removed request for Felicious August 14, 2025 15:56
@jero-odoo jero-odoo removed the 1 label Aug 14, 2025
@jero-odoo jero-odoo force-pushed the 18.0-settings-email-loops-jero branch from a0590ad to 575aada Compare August 14, 2025 18:10
@jero-odoo jero-odoo added the 2 label Aug 14, 2025
@jero-odoo jero-odoo requested a review from Felicious August 14, 2025 18:34
@jero-odoo
Copy link
Contributor Author

@Felicious this is ready for review now. Thank you for understanding!

Copy link
Contributor

@Felicious Felicious 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 to me, @jero-odoo !

@robodoo r+

Comment on lines +252 to +253
Incoming mail servers are created by going to :menuselection:`Settings --> Technical --> Emails:
Incoming Mail Servers`.
Copy link
Contributor

Choose a reason for hiding this comment

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

note: we don't do the formatting [heading section]: [dropdown heading] anymore, and only include the heading in the menuselection.

However, since the technical menu is a papyrus scroll, i think we should allow this exception for it (:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants