rokwire / illinois-app

Source code repository of "Illinois" App - the official mobile app of the University of Illinois.
https://app.illinois.edu/
Apache License 2.0
23 stars 20 forks source link

[USABILITY] More meaningful notifications for Group posts #4050

Closed jake-fava closed 3 months ago

jake-fava commented 4 months ago

All notifications for group posts are identical and generic. To make group notifications more meaningful to the user, all post notifications should include the name of the post author and a preview of the post title.

New post notification to all group members: Group - Illini Squirrel Enthusiasts Jake Fava posted “!!Squirrel Spotted!!” Time Stamp

In the case of a longer title truncate the post with …: Group - Illini Squirrel Enthusiasts Jake Fava posted “Contest: Squirrel drawing and…” Time Stamp

Notification for a new reply to a post should only go to the group members who are active in that particular post thread. See related ticket #2938

The reply notification should include the author’s name who replied and a preview of their reply: Group - Illini Squirrel Enthusiasts Jake Fava replied to your post “this is amazing!” Time Stamp

And if the reply is to a post the user didn't create themselves (again, see #2938): Group - Illini Squirrel Enthusiasts Jake Fava replied to a post “this is amazing!” Time Stamp

@vburgett

v 5.1.34

dobromirdobrev commented 3 months ago

@jake-fava , @vburgett , This should be handled on the backend as the relevant issue is closed. Could you please verify it on dev environment when you have a chance? Thanks.

jake-fava commented 3 months ago

A couple changes still need to be made:

Please use the double quotation marks ("") instead of single quotation marks ('') If the post is longer than 250 characters, truncate the notification and end it with an ellipses (...) Screenshot_20240626-121330 Screenshot_20240626-121342

dobromirdobrev commented 3 months ago

@jake-fava ,

Truncating message body is implemented in version 6.0.15 of the Illinois app. We will have to wait #457 to be implemented for the quotation marks.