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

[stable14] Fix the warning appearing in the admin section when mail_smtpmode is not configured #12529

Merged
merged 3 commits into from
Nov 19, 2018

Conversation

MorrisJobke
Copy link
Member

Backport of #12401

I would port this to 14.0.4 (as we need an RC 2 for #12502 anyways) and because this blocks that stable14 is shipped via snap by @kyrofa. Also this is a change of one default attribute in the setup check and has little impact on the overall instance.

Copy link
Member

@rullzer rullzer left a comment

Choose a reason for hiding this comment

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

🐘

Copy link
Member

@ChristophWurst ChristophWurst left a comment

Choose a reason for hiding this comment

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

Makes sense

@ChristophWurst ChristophWurst added 4. to release Ready to be released and/or waiting for tests to finish and removed 3. to review Waiting for reviews labels Nov 19, 2018
@MorrisJobke MorrisJobke mentioned this pull request Nov 19, 2018
pachulo and others added 3 commits November 19, 2018 15:49
Signed-off-by: Kyle Fazzari <kyrofa@ubuntu.com>
Signed-off-by: Morris Jobke <hey@morrisjobke.de>
@MorrisJobke MorrisJobke force-pushed the backport/12401/php-warning branch from f719750 to 511eade Compare November 19, 2018 14:49
Copy link

@JanDragon JanDragon left a comment

Choose a reason for hiding this comment

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

Code looks good.

Copy link
Member

@kyrofa kyrofa left a comment

Choose a reason for hiding this comment

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

Just tested, works as expected 👍 . Thanks @MorrisJobke!

@rullzer rullzer merged commit b133ca3 into stable14 Nov 19, 2018
@rullzer rullzer deleted the backport/12401/php-warning branch November 19, 2018 18:43
@MorrisJobke MorrisJobke mentioned this pull request Nov 22, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
4. to release Ready to be released and/or waiting for tests to finish
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants