aws-ses: add page #7136
aws-ses: add page #7136
Conversation
This comment has been minimized.
This comment has been minimized.
pages/common/aws-ses.md
Outdated
| @@ -0,0 +1,37 @@ | |||
| # aws ses | |||
|
|
|||
| > CLI for AWS SES. | |||
There was a problem hiding this comment.
Can we expand on SES please? Amazon Web Services provide a truly staggering number of services, so it's difficult to know which service this it at first glance.
There was a problem hiding this comment.
Good point! I'll update this to describe what the acronym stands for, thanks for the comment.
pages/common/aws-ses.md
Outdated
| > High-scale inbound and outbound cloud email service. | ||
| > More information: <https://awscli.amazonaws.com/v2/documentation/api/latest/reference/ses/index.html>. | ||
|
|
||
| - Show help for specific SES subcommand: |
There was a problem hiding this comment.
Perhaps this will read better?
| - Show help for specific SES subcommand: | |
| - Show help for a specific SES subcommand: |
Also, this example should be at the bottom of the page.
pages/common/aws-ses.md
Outdated
|
|
||
| - Send email: | ||
|
|
||
| `aws ses send-email --from {{from_address}} --destination "ToAddresses={{addresses}}" --message "Subject={Data={{subject_text}},Charset=utf8},Body={Text={Data={{body_text}},Charset=utf8},Html={Data={{}},Charset=utf8}}"` |
There was a problem hiding this comment.
Is the {{}} in Html={Data={{}}, a typo?
There was a problem hiding this comment.
Oops, yes it is, I'll add the placeholder text. It should be same as the other Data arguments.
navarroaxel
added
the
hacktoberfest-accepted
Apply suggestions from code review to improve readability. Co-authored-by: Starbeamrainbowlabs <sbrl@starbeamrainbowlabs.com>
This comment has been minimized.
This comment has been minimized.
Move help command to bottom of the list as per suggestion in PR review.
shermanhui
force-pushed
the
aws-ses-add-page
branch
from
b1872a3
to
b50a28c
Compare
There was a problem hiding this comment.
Thanks, @shermanhui!
Can't tell if you force-pushed or whether the view changes button is broken on my end haha
|
@sbrl I think I might have force-pushed after the bot indicated I missed a new line at the end of the file 😅 I didn't think it warranted a separate commit merely to add a newline at the end of file 🙏🏻 |
navarroaxel
removed
the
review needed
common/,linux/, etc.)Version of the command being documented (if known):
version 2010-12-01