GitHub PullRequestReviewCommentEvent at 2014-06-04 03:27:46
Repository: edx-platform
Actor: brianhw
Payload: {"comment":{"url":"https://api.github.com/repos/edx/edx-platform/pulls/comments/13369781","id":13369781,"diff_hunk":"@@ -207,18 +230,19 @@ To produce the Email Task History report:\n        :width: 800\n        :alt: A tabular report with a row for each message sent and columns for requester, date and time submitted, duration, state, task status, and task progress.\n \n-For messages with a State of \"Success\", the Task Progress can show an\n+For messages with a **State** of Success, the **Task Progress** can show an\n informational message such as \"Message successfully emailed for 13457 recipients\n (skipping 29) (out of 13486)\". To interpret this message, note that:\n \n * The first number indicates the number of messages sent to the selected\n   recipients.\n+\n * The second number (\"skipping\") indicates the number of enrolled users who will\n-  not be sent the message. This count includes students have opted out of\n+  not be sent the message. This count includes students who have opted out of\n   receiving course email messages and students who have not yet activated their\n   user accounts.\n+\n * The final number (\"out of\") indicates the number of users in the set of\n   recipients you selected who were enrolled in the course when you sent the\n   email message. For email messages addressed to a large number of users, the\n-  number of enrolled students can change while the message is queued for\n-  processing.\n+  number of enrolled students can change while the message is being processed.","path":"docs/en_us/course_authors/source/releasing_course/course_launching.rst","position":120,"original_position":120,"commit_id":"c7b685abfaf9ab2d8b8a7b6542abbfee52eb6c90","original_commit_id":"c7b685abfaf9ab2d8b8a7b6542abbfee52eb6c90","user":{"login":"brianhw","id":2534095,"avatar_url":"https://avatars.githubusercontent.com/u/2534095?","gravatar_id":"c22acd7358f58913f51f206d93cfdfa8","url":"https://api.github.com/users/brianhw","html_url":"https://github.com/brianhw","followers_url":"https://api.github.com/users/brianhw/followers","following_url":"https://api.github.com/users/brianhw/following{/other_user}","gists_url":"https://api.github.com/users/brianhw/gists{/gist_id}","starred_url":"https://api.github.com/users/brianhw/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/brianhw/subscriptions","organizations_url":"https://api.github.com/users/brianhw/orgs","repos_url":"https://api.github.com/users/brianhw/repos","events_url":"https://api.github.com/users/brianhw/events{/privacy}","received_events_url":"https://api.github.com/users/brianhw/received_events","type":"User","site_admin":false},"body":"The final number indicates the number of users who were enrolled *and* who had activated their email accounts.  This is why this number does not always match the total enrollment for the course, as it may appear elsewhere on the instructor dashboard.  \r\n\r\nI'm not sure about the assertion that the number of enrolled students can change.  Once the subtasks are defined, the number returned as the \"out of\" number should not change while the emails are being sent.  There should be no change in anything that they see.  It is true that the number may change while the subtasks are being defined, but I don't think any counts are displayed yet.","created_at":"2014-06-04T03:27:46Z","updated_at":"2014-06-04T03:27:46Z","html_url":"https://github.com/edx/edx-platform/pull/3975#discussion_r13369781","pull_request_url":"https://api.github.com/repos/edx/edx-platform/pulls/3975","_links":{"self":{"href":"https://api.github.com/repos/edx/edx-platform/pulls/comments/13369781"},"html":{"href":"https://github.com/edx/edx-platform/pull/3975#discussion_r13369781"},"pull_request":{"href":"https://api.github.com/repos/edx/edx-platform/pulls/3975"}}}}