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

Update mail server configuration section #110

Merged
merged 3 commits into from
Nov 10, 2021
Merged

Update mail server configuration section #110

merged 3 commits into from
Nov 10, 2021

Conversation

javierm
Copy link
Member

@javierm javierm commented Nov 10, 2021

References

Background

We don't recommend using a mail server in the development environment; in that environment, mails are opened in the browser. Even so, the documentation about configuring an email server referenced the development environment.

Objectives

  • Update the mail server configuration so it's clear that it applies to production environments
  • Update the mail server configuration so it uses the secrets file instead of the environment file

@javierm javierm self-assigned this Nov 10, 2021
@javierm javierm changed the title Mailserver Update mail server configuration section Nov 10, 2021
@javierm javierm mentioned this pull request Nov 10, 2021
@taitus taitus self-assigned this Nov 10, 2021
Mail server configuration now is done on `config/secrets.yml` and thus
documentation should be updated accordingly.
@javierm javierm merged commit a656cac into master Nov 10, 2021
@javierm javierm deleted the mailserver branch November 10, 2021 12:39
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Update email configuration documentation, please. Update documentation for email in development
3 participants