Community Notifications Changes (prod)

topcoder-platform / tc-notifications

4 stars 15 forks source link

Closed sachin-maheshwari closed 4 years ago

sachin-maheshwari commented 4 years ago

Covers following issue/features -

Avoid multiple calls for member api and group api
Fix for duplicate notifications (challenge api)
Stopping notification for "A new submission is uploaded" events (submission.notification.create)
Changes as per described at topcoder-platform/community-app#4114
Public group logic changes - now excluding group like ["!CS", "!Zurich", "!Topgear"]

sachin-maheshwari commented 4 years ago

@maxceem can i take your changes for Production (planning next week) ?

maxceem commented 4 years ago

@sachin-maheshwari yes, I've just removed a redundant constant which we don't use anymore, to keep it clear, as per https://github.com/appirio-tech/connect-app/issues/3505

sachin-maheshwari commented 4 years ago

@maxceem 👍