Add Notifications release note #1358
Conversation
ayub02
left a comment
ayub02
left a comment
There was a problem hiding this comment.
I suggested some small changes. Thanks for creating this PR.
| @@ -0,0 +1,162 @@ | |||
| .. _Ulmo Notifications: | |||
|
|
|||
| Introducing Platform Notifications | |||
There was a problem hiding this comment.
| Introducing Platform Notifications | |
| Notifications For Learning Activity |
There was a problem hiding this comment.
@ayub02 - this is meant to be more of a marketing-focused note. I don't think your suggestion is as exciting/grabs attention as my suggestion. Would you like to suggest something more marketing-focused for this, if you don't like my suggestion?
There was a problem hiding this comment.
Maybe "Introducing Activity Notifications"? I think "Introducing Learning Activity Notifications" is a bit long.
There was a problem hiding this comment.
I didn't know this was marketing focused. Please proceed as you suggested. I changed it because i was looking for notifications in the list but took a few seconds to realize that notifications had 2 words before it.
| reviewing activity, and understanding delivery settings, please refer to the | ||
| :ref:`Notifications & Preferences`. | ||
|
|
||
| For site operators, configuration options are available in the :ref:`Operator release notes <Ulmo operators>`. |
There was a problem hiding this comment.
TODO: Change this link to the site ops how-to when #1360 is merged
Apply suggestions from code review Co-authored-by: Aamir Ayub <40331925+ayub02@users.noreply.github.com>
sarina
force-pushed
the
sarina/ulmo-notifications
branch
from
e7fd51d to
1136cb9
Compare
2 participants
Uh oh!
There was an error while loading. Please reload this page.