-
Notifications
You must be signed in to change notification settings - Fork 369
Conversation
a348693
to
9318f8c
Compare
4295b0e
to
900acb9
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please include also changes in docs. It's difficult to catch up with docs at the later stage.
...ure/src/main/java/org/springframework/cloud/aws/autoconfigure/mail/SesAutoConfiguration.java
Show resolved
Hide resolved
94969e4
to
894589e
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great work @eddumelendez! When you merge it, please only write longer commit message explaining what and why are we doing here.
I think it would be great if we add documentation together with PRs with code changes. |
894589e
to
ce3403e
Compare
@maciejwalkowiak I have rebase the branch and update the docs. Could you review it again, please? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Apologies for long delay.
New modules are introduced: * spring-cloud-aws-ses * spring-cloud-starter-aws-ses Closes spring-atticgh-664
bc71d0b
to
c4683fe
Compare
Thanks for this, I've just had to sort out SES (in Spring Cloud AWS) and it's been a little tricky, hopefully this will make it allot easier in future for everyone. |
New modules are introduced: * spring-cloud-aws-ses * spring-cloud-starter-aws-ses References to `SimpleEmail*` have been deprecated. Closes spring-attic/spring-cloud-aws#664
Closes gh-664