Note: For guidelines and instructions pages linked below, log in with user name / password: drupal/drupal.

Task for this issue

Copy edit the following topic:

Title: Keeping Track of Updates
Section: Security and Maintenance
File name: security-announce.txt

Look for:
- Typographical errors
- Spelling errors
- Grammar and usage errors
- Punctuation problems
- Adherence to the project's editing guidelines: https://userguide_new-drupal.dev.devdrupal.org/guidelines/guidelines-edi... (access: drupal / drupal )
- Adherence to the project's writing guidelines: https://userguide_new-drupal.dev.devdrupal.org/guidelines/guidelines-wri... (access: drupal / drupal )
- Formatting problems in the output. The User guide output can be found on: https://userguide_new-drupal.dev.devdrupal.org/d8guide/en/index.html (access: drupal / drupal )

Note that the "single-issue" guidelines in the Editing guidelines page, as well as the entire Writing guidelines page, should have already been enforced. But it is still a good idea to at least check them quickly.

Instructions

See https://userguide_new-drupal.dev.devdrupal.org/guidelines/instructions.h... -- but skip the section on finding a task to do, since you have selected this one.

Checklist for this task

- Is a patch file provided? If only one or a few topics were edited, you can instead attach the updated source file(s).
- Is the edit complete and correct?
- Is the work free from additional edits outside the scope of the provided guidelines or task?
- Is attribution provided? See https://userguide_new-drupal.dev.devdrupal.org/guidelines/guidelines.htm...

Comments

jhodgdon created an issue. See original summary.

swarad07’s picture

Assigned: Unassigned » swarad07
swarad07’s picture

Status: Active » Needs review

No issues found here too. I verified the instructions that are mentioned for adding an email in order to receive notifications about Drupal updates, they are correct!

jhodgdon’s picture

Status: Needs review » Fixed

Excellent, thanks!

jhodgdon’s picture

whoops, forgot issue credit

jojyja’s picture

Assigned: swarad07 » jojyja
Status: Fixed » Active
jojyja’s picture

Assigned: jojyja » Unassigned
Status: Active » Needs review
StatusFileSize
new967 bytes

I looked this topic over again and found some minor edits. Here's a patch.

jhodgdon’s picture

Status: Needs review » Needs work

Thanks for checking this again! I agree that the navigation instructions needed an update. However:

+++ b/source/en/security-announce.txt
@@ -23,10 +23,7 @@ updates.
+* In the _Manage_ administrative menu, navigate to _Reports_ > _Available updates_ > _Settings_. On the _Update Manager settings_ page, fill in the email addresses to be notified when updates are available. You can also specify whether you want daily or weekly updates. Click _Save configuration_.

I found the navigation instructions you put in this patch to be a bit confusing, where it says "On the Update manager settings page...". I felt like it was saying I would have to navigate to reports > available updates > settings, and then find the Update Manager settings page from there. However, it's just the name of the page that you end up on... So, I think it would be clearer of we just took out that clause, so it says: "Navigate to (whatever). Enter the email address to be ..." [also, "enter" is probably better than "fill in"].

Thoughts?

jojyja’s picture

I agree with you. I will take that out.

jojyja’s picture

Status: Needs work » Needs review
StatusFileSize
new926 bytes

Here is the patch featuring the update.

jhodgdon’s picture

Status: Needs review » Fixed

Thanks -- that looks great! Committed.

  • jhodgdon committed 30cb5d4 on 8.x-0.x authored by jojyja
    Issue #2754579 by jojyja: Copy edit security-announce.txt
    

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.