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

PROD-8414 - Email notifications for secondary replies not working #4616

Open
wants to merge 2 commits into
base: release
Choose a base branch
from

Conversation

jitendrabanjara1991
Copy link
Collaborator

Jira Issue:

https://buddyboss.atlassian.net/browse/PROD-8414

General Note

Keep all conversations related to this PR in the associated Jira issue(s). Do NOT add comment on this PR or edit this PR’s description.

Notes to Developer

  • Ensure the IDs (i.e. PROD-1) of all associated Jira issues are reference in this PR’s title
  • Ensure that you have achieved the Definition of Done before submitting for review
  • When this PR is ready for review, move the associate Jira issue(s) to “Needs Review” (or “Code Review” for Dev Tasks)

Notes to Reviewer

  • Ensure that the Definition of Done have been achieved before approving a PR
  • When this PR is approved, move the associated Jira issue(s) to “Needs QA” (or “Approved” for Dev Tasks)

…ity_comment_posted', 'bb_activity_comment_at_name_send_emails', 12, 3 ); was set to priority 12, causing it to trigger after notifications were already added. This was creating an issue with mentions.
* release:
Update: Grunt
Update: Grunt string-replace
Update: Version name update
Added change log
grunt
PROD-8551 - fix cross-site scripting vulnerability for the forums
PROD-8551 - fix cross-site scripting vulnerability
PROD-8245 - Fixed - The invites table in the back end of the site does not sort properly
PROD-8510 - Fixed - The group type with the setting to hide the groups from the Group Directory hides those groups from the Backend groups directory
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.

1 participant