Skip to content

Fix typo in configuring-email-for-notifications.md#3094

Merged
shati-patel merged 6 commits into
mainfrom
shati-patel-patch-1
Jan 26, 2021
Merged

Fix typo in configuring-email-for-notifications.md#3094
shati-patel merged 6 commits into
mainfrom
shati-patel-patch-1

Conversation

@shati-patel

@shati-patel shati-patel commented Jan 23, 2021

Copy link
Copy Markdown
Contributor

Why:

There's a typo ("incom") and unnecessary newline in this article.

What's being changed:

Staging Source Notes
Configuring email for notifications Source Remove typo and newline

Check off the following:

@welcome

welcome Bot commented Jan 23, 2021

Copy link
Copy Markdown

Thanks for opening this pull request! A GitHub docs team member should be by to give feedback soon. In the meantime, please check out the contributing guidelines.

@shati-patel
shati-patel marked this pull request as ready for review January 23, 2021 14:15
@shati-patel shati-patel added content This issue or pull request belongs to the Docs Content team core labels Jan 25, 2021

@janiceilene janiceilene left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This looks great @shati-patel! :shipit:

@shati-patel
shati-patel merged commit 141da3e into main Jan 26, 2021
@shati-patel
shati-patel deleted the shati-patel-patch-1 branch January 26, 2021 10:57
@github-actions

Copy link
Copy Markdown
Contributor

Thanks very much for contributing! Your pull request has been merged 🎉 You should see your changes appear on the site in approximately 24 hours.

@Realest1Re Realest1Re mentioned this pull request Nov 20, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

content This issue or pull request belongs to the Docs Content team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants