Skip to content
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

18.0 mig mail outbound static #1564

Draft
wants to merge 44 commits into
base: 18.0
Choose a base branch
from

Conversation

matteoopenf
Copy link

No description provided.

lasley and others added 30 commits January 29, 2025 15:31
* Allow for the configuration of the FROM header for outbound emails, and move the existing from header to Sender as per RFC-2822
Fixing and extending tests
Add setup
use sudo to get ir.config_parameter, otherwise when normal users try to
send email they get AccessError
- fix misspellings errors in documentation
- add unit test to avoid github checkers red status in codecov tests.
- add validation of email from and domain whitelist fields.
- update documentation remove roadmap for already updated functionalities.
- update translations
Since 15.0, smtp info is prepared in _prepare_email_message() which is called
from send_email(), and the smtp info manipulation should be done in this method
instead of in send_email().
@matteoopenf matteoopenf force-pushed the 18.0-mig-mail_outbound_static branch 4 times, most recently from acfb4dc to 48dc81b Compare January 29, 2025 15:14
@matteoopenf matteoopenf force-pushed the 18.0-mig-mail_outbound_static branch from 48dc81b to 1e66074 Compare January 29, 2025 15:34
@pedrobaeza
Copy link
Member

This should go to the new repo OCA/mail.

@matteoopenf
Copy link
Author

This should go to the new repo OCA/mail.

with or without history?

@pedrobaeza
Copy link
Member

With commit history. For doing that, you just need to add both remotes, and change the command to point the source to the other remote.

@matteoopenf
Copy link
Author

With commit history. For doing that, you just need to add both remotes, and change the command to point the source to the other remote.

Ok

Have you also some idea how can I fix the test?
Thank you

@pedrobaeza
Copy link
Member

Sorry, I don't use this module.

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.