Skip to content

[FIX] ecommerce: require module install for email queue optimization #12389

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

AntoineVDV
Copy link
Collaborator

@AntoineVDV AntoineVDV commented Mar 12, 2025

In version 17.0 to 18.2, it is required to install the sale_async_emails module to enable the email queue optimization.

This commit also rephrases slightly the step to enable the cron because it should already be enabled, unless the user disabled it manually. Indeed, the cron is created enabled in versions 17.0 to 18.2, while it is created disabled but automatically enabled when toggling the sale.async_emails system parameter in master.

task-3872792

In version 17.0 to 18.2, it is required to install the
`sale_async_emails` module to enable the email queue optimization.

This commit also rephrases slightly the step to enable the cron because
it should already be enabled, unless the user disabled it manually.
Indeed, the cron is created enabled in versions 17.0 to 18.2, while it
is created disabled but automatically enabled when toggling the
`sale.async_emails` system parameter.

task-3872792
@AntoineVDV AntoineVDV requested a review from auva-odoo March 12, 2025 16:17
@robodoo
Copy link
Collaborator

robodoo commented Mar 12, 2025

Pull request status dashboard

@C3POdoo C3POdoo requested a review from a team March 12, 2025 16:18
Copy link
Contributor

@auva-odoo auva-odoo left a comment

Choose a reason for hiding this comment

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

Thank you @AntoineVDV !

@robodoo r+ up to saas-18.2

@robodoo
Copy link
Collaborator

robodoo commented Mar 13, 2025

Forward-porting to 'saas-18.2'.

@AntoineVDV
Copy link
Collaborator Author

@robodoo up to master

I'll remove the module mention in master but keep the new phrasing since it applies in 18.3+ too.

@robodoo
Copy link
Collaborator

robodoo commented Mar 13, 2025

Forward-porting to 'master'.

@AntoineVDV
Copy link
Collaborator Author

@robodoo fw=skipmerge

@robodoo
Copy link
Collaborator

robodoo commented Mar 13, 2025

Starting forward-port. Not waiting for merge to create followup forward-ports.

robodoo pushed a commit that referenced this pull request Mar 13, 2025
In version 17.0 to 18.2, it is required to install the
`sale_async_emails` module to enable the email queue optimization.

This commit also rephrases slightly the step to enable the cron because
it should already be enabled, unless the user disabled it manually.
Indeed, the cron is created enabled in versions 17.0 to 18.2, while it
is created disabled but automatically enabled when toggling the
`sale.async_emails` system parameter.

task-3872792

closes #12389

Signed-off-by: Audrey Vandromme (auva) <auva@odoo.com>
robodoo pushed a commit that referenced this pull request Mar 13, 2025
In version 17.0 to 18.2, it is required to install the
`sale_async_emails` module to enable the email queue optimization.

This commit also rephrases slightly the step to enable the cron because
it should already be enabled, unless the user disabled it manually.
Indeed, the cron is created enabled in versions 17.0 to 18.2, while it
is created disabled but automatically enabled when toggling the
`sale.async_emails` system parameter.

task-3872792

closes #12389

Signed-off-by: Audrey Vandromme (auva) <auva@odoo.com>
@robodoo robodoo closed this Mar 13, 2025
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.

3 participants