Skip to content

[16.0][ADD] repair_custom_sender: Allows configuring a custom sender email for repair orders#157

Closed
nicolas-delbovier-acsone wants to merge 1 commit into
OCA:16.0from
acsone:16.0-add-repair_custom_sender
Closed

[16.0][ADD] repair_custom_sender: Allows configuring a custom sender email for repair orders#157
nicolas-delbovier-acsone wants to merge 1 commit into
OCA:16.0from
acsone:16.0-add-repair_custom_sender

Conversation

@nicolas-delbovier-acsone
Copy link
Copy Markdown

@nicolas-delbovier-acsone nicolas-delbovier-acsone commented Apr 8, 2026

Use case

By default, Odoo sets the sender as the user who clicks "Send" on the wizard. This module allows setting a centralized generic email (e.g., repairs@company.com) for all repair communications.

Configuration

image

Technical note

I first tried to implement this by passing "default_email_from" in context but it seems this approach does not work as the email_from field gets overwritten later down the line. If you have an idea of how to make this work / why this does not work, I'd be happy ton hear it (I believe this would be a more elegant solution to ensure the wizard gets created with the good values at creation instead of having to overwrite)

@nicolas-delbovier-acsone nicolas-delbovier-acsone force-pushed the 16.0-add-repair_custom_sender branch from 45f4667 to cc4221d Compare April 14, 2026 15:01
@OCA-git-bot OCA-git-bot added series:16.0 mod:repair_custom_sender Module repair_custom_sender labels Apr 14, 2026
@nicolas-delbovier-acsone
Copy link
Copy Markdown
Author

Closing Rationale

Closing this PR to rework the implementation.

Key findings:

  • The From header is easily managed via standard Mail Templates.
  • However, Odoo's _notify_get_reply_to logic forces the Reply-To back to the catchall (in case of a mail.thread object), even when templates specify otherwise.

The objective is to ensure Reply-To remains synced with the custom From address. I will submit a new PR that specifically targets this.

@nicolas-delbovier-acsone
Copy link
Copy Markdown
Author

See #162

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

mod:repair_custom_sender Module repair_custom_sender series:16.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants