Skip to content

Conversation

@hghorbel
Copy link
Contributor

@hghorbel hghorbel commented May 8, 2020

No description provided.

@hghorbel hghorbel requested a review from plamarque May 8, 2020 21:08
Copy link
Member

@plamarque plamarque left a comment

Choose a reason for hiding this comment

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

language improvements suggested.
I also commented on suspicious content that seeems copy_paste or old content that needs to be removed.

your See all on the bottom notifications drawer :

|settings7|

Copy link
Member

Choose a reason for hiding this comment

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

duPlicate content


|settings8|

.. tip:: When you are not on the platform web page and you receive On-site notifications,
Copy link
Member

Choose a reason for hiding this comment

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

duplicate content


.. _mobilePushNotifs:

**Mobile push notifications**
Copy link
Member

Choose a reason for hiding this comment

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

We should indicate that push notifications channel will be available if the administrator has enabled and setup push notifs and then link to the admin screen where this can be configured

hghorbel and others added 18 commits May 9, 2020 14:31
Co-authored-by: Patrice Lamarque <plamarque@exoplatform.com>
Co-authored-by: Patrice Lamarque <plamarque@exoplatform.com>
Co-authored-by: Patrice Lamarque <plamarque@exoplatform.com>
Co-authored-by: Patrice Lamarque <plamarque@exoplatform.com>
Co-authored-by: Patrice Lamarque <plamarque@exoplatform.com>
Co-authored-by: Patrice Lamarque <plamarque@exoplatform.com>
Co-authored-by: Patrice Lamarque <plamarque@exoplatform.com>
Co-authored-by: Patrice Lamarque <plamarque@exoplatform.com>
Co-authored-by: Patrice Lamarque <plamarque@exoplatform.com>
Co-authored-by: Patrice Lamarque <plamarque@exoplatform.com>
Co-authored-by: Patrice Lamarque <plamarque@exoplatform.com>
Co-authored-by: Patrice Lamarque <plamarque@exoplatform.com>
Co-authored-by: Patrice Lamarque <plamarque@exoplatform.com>
Co-authored-by: Patrice Lamarque <plamarque@exoplatform.com>
Co-authored-by: Patrice Lamarque <plamarque@exoplatform.com>
Co-authored-by: Patrice Lamarque <plamarque@exoplatform.com>
Co-authored-by: Patrice Lamarque <plamarque@exoplatform.com>
Co-authored-by: Patrice Lamarque <plamarque@exoplatform.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants