Notifications for Comments

[gmeben]

Users are not notified when users add comments on submissions. This means users may not be aware of discussion activity on submissions they are reviewing.

Add notifications for comments added to a submission based on role and proximity to newly added comments.

Recipients

Newly Added Inline Comments

• Submitters
• Review Coordinators

Newly Added Inline Comment Replies

• Submitters
• Reviewers with an inline comment reply of the parent inline comment
• Review Coordinators

Newly Added Overall Comment Replies

• Submitters
• Reviewers with an overall comment reply of the parent overall comment
• Review Coordinators

Review Checklist

• Submitters are notified of comments
• Reviewers are notified of comments
• Review Coordinators are notified of comments
• Necessary backend unit testing has been added

Accessibility Checklist

• No new color contrast issues are introduced
• Changed text has a font size of at least 16px
• Added text uses plain language

[gmeben]

The questions for this issue are:

1. Do we agree on the specified recipients of these notifications?
2. Do we agree on the criteria of these notifications: any comment added to a submission regardless of thread proximity?

(Update) Answers:

1. No
2. No

The recipients and criteria needed to be more selective. The description has been updated to reflect this.

[gmeben]

Feature Request: https://feedback.pilcrow.dev/posts/8/way-to-denote-an-updated-comment

[sattler9]

Hey team! Please add your planning poker estimate with Zenhub @gmeben @vilagshe @wreality

[gmeben]

I forgot to document from our team discussion on this prior to today that there was a desire to make these notifications in-app only, not email.