Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: endorsed post notification audiance updated #34513

Merged
merged 1 commit into from
Apr 22, 2024

Conversation

AhtishamShahid
Copy link
Contributor

@AhtishamShahid AhtishamShahid commented Apr 16, 2024

Description

Updated endorsed response notification logic

  • If the endorsed response and thread have the same user, only send one notification.
  • if the response is endorsed by the creator or response, do not send a notification.

Ticket

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

fix: updated endorsed notification logic

fix: updated endorsed notification logic
@AhtishamShahid AhtishamShahid merged commit 5b1d52d into master Apr 22, 2024
66 checks passed
@AhtishamShahid AhtishamShahid deleted the ahtisham/INF-1311 branch April 22, 2024 08:09
@edx-pipeline-bot
Copy link
Contributor

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

@edx-pipeline-bot
Copy link
Contributor

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

KyryloKireiev pushed a commit to raccoongang/edx-platform that referenced this pull request Apr 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants