edx-platform icon indicating copy to clipboard operation
edx-platform copied to clipboard
verified publisher icon openedx

feat: added notification grouping

Open muhammadadeeltajamul opened this issue 1 year ago • 2 comments

Description :

Added grouping feature for new comment notifications

Ticket

https://2u-internal.atlassian.net/browse/INF-1472

Screenshot 2024-09-05 at 2 30 17 PM

muhammadadeeltajamul avatar Aug 23 '24 12:08 muhammadadeeltajamul

Author pronouns are not working for both grouped and individual notifications.

Screenshot 2024-09-08 at 12 31 52 AM Screenshot 2024-09-08 at 12 33 25 AM Screenshot 2024-09-08 at 12 39 25 AM

eemaanamir avatar Sep 09 '24 05:09 eemaanamir

The following edge cases need to be handled:

  1. When the comment user is the same as the response user and the notification is grouped the response user's name should replace the pronoun.
  2. When the same user adds two comments under the same response, the notification content displays the user's name twice. (highlighted in purple)
Screenshot 2024-09-11 at 9 13 22 PM Screenshot 2024-09-11 at 9 18 40 PM

eemaanamir avatar Sep 11 '24 16:09 eemaanamir

2U Release Notice: This PR has been deployed to the edX staging environment in preparation for a release to production.

edx-pipeline-bot avatar Oct 02 '24 09:10 edx-pipeline-bot

2U Release Notice: This PR has been deployed to the edX production environment.

edx-pipeline-bot avatar Oct 02 '24 10:10 edx-pipeline-bot

2U Release Notice: This PR has been deployed to the edX production environment.

edx-pipeline-bot avatar Oct 02 '24 10:10 edx-pipeline-bot