Skip to content

Commit dba95da

Browse files
authored
Merge pull request #17351 from github/3059-remove-send-from-author
Remove `Send from author` in notification articles
2 parents faa156f + 4996deb commit dba95da

8 files changed

Lines changed: 6 additions & 12 deletions

File tree

-3.04 KB
Loading
15.7 KB
Loading
-12.8 KB
Binary file not shown.
-10.9 KB
Binary file not shown.
8.53 KB
Loading

content/admin/configuration/configuring-email-for-notifications.md

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -44,12 +44,6 @@ settings to allow incoming emails](#configuring-dns-and-firewall-settings-to-all
4444
- In the **Authentication** dropdown, choose the type of encryption used by your SMTP server.
4545
- In the **No-reply email address** field, type the email address to use in the From and To fields for all notification emails.
4646

47-
{% note %}
48-
49-
**Note:** If you select the **Send from author** checkbox in a repository’s **Services** email webhook, outbound email for that repository will send from the author and not from the no-reply email address. For more information, see "[About email notifications for pushes to your repository](/github/administering-a-repository/about-email-notifications-for-pushes-to-your-repository)."
50-
51-
{% endnote %}
52-
5347
6. If you want to discard all incoming emails that are addressed to the no-reply email address, select **Discard email addressed to the no-reply email address**.
5448
![Checkbox to discard emails addressed to the no-reply email address](/assets/images/enterprise/management-console/discard-noreply-emails.png)
5549
7. Under **Support**, choose a type of link to offer additional support to your users:

content/github/administering-a-repository/about-email-notifications-for-pushes-to-your-repository.md

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -34,10 +34,10 @@ You can filter email notifications you receive for pushes to a repository. For m
3434
{% data reusables.repositories.sidebar-notifications %}
3535
5. Type up to two email addresses, separated by whitespace, where you'd like notifications to be sent. If you'd like to send emails to more than two accounts, set one of the email addresses to a group email address.
3636
![Email address textbox](/assets/images/help/settings/email_services_addresses.png)
37-
6. If you operate your own server, you can verify the integrity of emails via the **Secret** token. This token is sent with the email as the `Approved` header. If the `Approved` header matches the token you sent, you can trust that the email is from {% data variables.product.product_name %}.
38-
![Email secret textbox](/assets/images/help/settings/email_services_token.png)
39-
7. Click **Save settings**.
40-
![Save settings button](/assets/images/help/settings/save_notification_settings.png)
37+
1. If you operate your own server, you can verify the integrity of emails via the **Approved header**. The **Approved header** is a token or secret that you type in this field, and that is sent with the email. If the `Approved` header of an email matches the token, you can trust that the email is from {% data variables.product.product_name %}.
38+
![Email approved header textbox](/assets/images/help/settings/email_services_approved_header.png)
39+
7. Click **Setup notifications**.
40+
![Setup notifications button](/assets/images/help/settings/setup_notifications_settings.png)
4141

4242
### Further reading
4343
{% if currentVersion == "free-pro-team@latest" or currentVersion == "github-ae@latest" or currentVersion ver_gt "enterprise-server@2.20" %}

tests/rendering/server.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -329,8 +329,8 @@ describe('server', () => {
329329
})
330330

331331
test('admin articles that link to Enterprise user articles have Enterprise user links', async () => {
332-
const $ = await getDOM(`${latestEnterprisePath}/admin/user-management/configuring-email-for-notifications`)
333-
expect($('article a[href*="about-email-notifications-for-pushes-to-your-repository"]').length).toBe(1)
332+
const $ = await getDOM(`${latestEnterprisePath}/admin/user-management/customizing-user-messages-for-your-enterprise`)
333+
expect($('article a[href*="about-writing-and-formatting-on-github"]').length).toBe(1)
334334
})
335335

336336
test('articles that link to external links that contain /articles/ are not rewritten', async () => {

0 commit comments

Comments
 (0)