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

NextCloud: Mention minimal SMTP configuration in README.md #2395

Merged
merged 1 commit into from
Jan 16, 2024

Conversation

rnestler
Copy link
Contributor

Looking at https://github.com/nextcloud/docker/blob/d1dbc77e0b5e851105cf8606942e74b7d21fe682/.config/smtp.config.php it is clear that SMTP_HOST, MAIL_FROM_ADDRESS and MAIL_DOMAIN must be set.

@LaurentGoderre LaurentGoderre changed the title Mention minimal SMTP configuration in README.md NextCloud: Mention minimal SMTP configuration in README.md Nov 28, 2023
@LaurentGoderre
Copy link
Member

@nextcloud

@LaurentGoderre
Copy link
Member

@rnestler Can you apply this to the content.md instead. The README gets auto-generated so your change would be lost

@rnestler
Copy link
Contributor Author

rnestler commented Dec 26, 2023

@LaurentGoderre fixed.

@tianon
Copy link
Member

tianon commented Jan 2, 2024

cc @J0WI @tilosp

@J0WI
Copy link
Contributor

J0WI commented Jan 9, 2024

@rnestler the canonical source for this would be in https://github.com/nextcloud/docker/blob/master/README.md. Would you like to open a PR there as well?

@rnestler
Copy link
Contributor Author

@J0WI Done. See nextcloud/docker#2144

@rnestler
Copy link
Contributor Author

PR to canonical source got merged.

@tianon tianon merged commit 5c90671 into docker-library:master Jan 16, 2024
5 checks passed
@rnestler rnestler deleted the patch-1 branch January 17, 2024 12:25
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.

4 participants