Skip to content

Commit eaa880e

Browse files
committed
feat: add Group Community TA audience for cohorted content_reported notifications
1 parent 042632e commit eaa880e

File tree

2 files changed

+6
-4
lines changed

2 files changed

+6
-4
lines changed

lms/djangoapps/discussion/rest_api/tests/test_discussions_notifications.py

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -51,10 +51,11 @@ def _assert_send_notification_called_with(self, mock_send_notification, expected
5151
'email_content': 'Thread body',
5252
})
5353
self.assertEqual(audience_filters, {
54-
'discussion_roles': ['Administrator', 'Moderator', 'Community TA']
54+
'discussion_roles': ['Administrator', 'Moderator', 'Community TA'],
55+
'cohort_for_group_tas': [1],
5556
})
56-
self.assertEqual(len(audience_filters), 1)
57-
self.assertEqual(list(audience_filters.keys()), ['discussion_roles'])
57+
self.assertEqual(len(audience_filters), 2)
58+
self.assertEqual(list(audience_filters.keys()), ['discussion_roles', 'cohort_for_group_tas'])
5859

5960
def test_send_reported_content_notification_for_response(self, mock_send_notification, mock_create_audience):
6061
"""

openedx/core/djangoapps/notifications/audience_filters.py

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -251,7 +251,8 @@ def apply_filters(self, user_ids, course_key, notification_type) -> list:
251251
)
252252
user_ids = getattr(self, filter_name)(user_ids, course)
253253
return user_ids
254-
254+
255+
255256
class GroupTAinCohortFilter(NotificationAudienceFilterBase):
256257
"""
257258
Returns Group Community TA (Group Moderator) user ids for the provided cohort IDs.

0 commit comments

Comments
 (0)