search

topcoder-platform / tc-notifications

4 stars 15 forks source link

Community Notifications Changes #183

Closed sachin-maheshwari closed 4 years ago

sachin-maheshwari commented 4 years ago

Covers following issue/features -

SathyaJayabal commented 4 years ago

verified on dev Stopping "A new submission is uploaded" events (submission.notification.create)

Tracked here : https://github.com/topcoder-platform/community-app/issues/4108

SathyaJayabal commented 4 years ago

verified on dev Duplicate notifications tracked here:https://github.com/topcoder-platform/community-app/issues/4105

SathyaJayabal commented 4 years ago

Verified on dev member api rate limit issue tracked here:https://github.com/topcoder-platform/tc-notifications/issues/182

SathyaJayabal commented 4 years ago

Public group logic changes - now excluding group like ["!CS", "!Zurich", "!Topgear"] tracked here : https://github.com/topcoder-platform/community-app/issues/4121

When the groups filter is empty, then all users should receive the notification. But users belonging to any group. (eg:cognitive) do not receive the notification.

updated https://github.com/topcoder-platform/community-app/issues/4121 with comments

SathyaJayabal commented 4 years ago

verified on dev Changes as per described at topcoder-platform/community-app#4114

SathyaJayabal commented 4 years ago

@sachin-maheshwari Public group logic changes - now excluding group like ["!CS", "!Zurich", "!Topgear"] tracked here : topcoder-platform/community-app#4121

verified on dev

sachin-maheshwari commented 4 years ago

@SathyaJayabal thank you! 👍