Skip to content
This repository has been archived by the owner on Apr 17, 2023. It is now read-only.

Expanded configuration for the mailer #1967

Merged
merged 1 commit into from
Sep 13, 2018
Merged

Conversation

mssola
Copy link
Collaborator

@mssola mssola commented Sep 12, 2018

The current configuration options were missing some important keys for
ActiveMailer on some scenarios.

Fixes #1876

Signed-off-by: Miquel Sabaté Solà [email protected]

@mssola
Copy link
Collaborator Author

mssola commented Sep 12, 2018

This should also do the trick for #1922.

@mssola
Copy link
Collaborator Author

mssola commented Sep 12, 2018

Tasks to be done:

@mssola mssola changed the title Expanded configuration for the mailer [WIP] Expanded configuration for the mailer Sep 12, 2018
Copy link
Contributor

@vitoravelino vitoravelino left a comment

Choose a reason for hiding this comment

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

So far looking good. 👍

mssola added a commit that referenced this pull request Sep 13, 2018
See #1967

Signed-off-by: Miquel Sabaté Solà <[email protected]>
@mssola mssola changed the title [WIP] Expanded configuration for the mailer Expanded configuration for the mailer Sep 13, 2018
@mssola
Copy link
Collaborator Author

mssola commented Sep 13, 2018

@vitoravelino you can review again 👍

vitoravelino
vitoravelino previously approved these changes Sep 13, 2018
The current configuration options were missing some important keys for
ActiveMailer on some scenarios.

Fixes SUSE#1876
Fixes SUSE#1922

Signed-off-by: Miquel Sabaté Solà <[email protected]>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants