Skip to content

Update AWS SES configuration doc#2105

Merged
georglauterbach merged 2 commits intodocker-mailserver:masterfrom
andrewlow:ses-doc-update
Jul 29, 2021
Merged

Update AWS SES configuration doc#2105
georglauterbach merged 2 commits intodocker-mailserver:masterfrom
andrewlow:ses-doc-update

Conversation

@andrewlow
Copy link
Copy Markdown
Contributor

@andrewlow andrewlow commented Jul 29, 2021

Description

AWS SES configuration changed, Instead of just a warning pointing people to the standard relay doc, I've re-written it with some guidance on gathering the right information from AWS and getting the configuration correct.

This doc update was requested as part of the discussion on this issue:
#2091 (comment)

Type of change

  • Improvement (non-breaking change that does improve existing functionality)

Checklist:

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation (README.md or the documentation under docs/)
  • If necessary I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes

(ok, this is just a doc change - most of the checklist is N/A)

Copy link
Copy Markdown
Member

@wernerfred wernerfred left a comment

Choose a reason for hiding this comment

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

LGTM, thanks

@wernerfred
Copy link
Copy Markdown
Member

@all-contributors add @andrewlow for documentation

@allcontributors
Copy link
Copy Markdown
Contributor

@wernerfred

I've put up a pull request to add @andrewlow! 🎉

@georglauterbach georglauterbach added area/documentation kind/improvement Improve an existing feature, configuration file or the documentation labels Jul 29, 2021
@georglauterbach georglauterbach added this to the v10.0.1 milestone Jul 29, 2021
@georglauterbach georglauterbach enabled auto-merge (squash) July 29, 2021 20:29
@georglauterbach georglauterbach merged commit 016d902 into docker-mailserver:master Jul 29, 2021
@github-actions
Copy link
Copy Markdown
Contributor

Documentation preview for this PR is ready! 🎉

Built with commit: 04cca9a

@andrewlow andrewlow deleted the ses-doc-update branch July 29, 2021 23:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/documentation kind/improvement Improve an existing feature, configuration file or the documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants