Skip to content

Fix notification targeting: don't notify unrelated reviewers#76

Merged
HamptonMakes merged 1 commit intomainfrom
fix-notification-targeting
Apr 3, 2026
Merged

Fix notification targeting: don't notify unrelated reviewers#76
HamptonMakes merged 1 commit intomainfrom
fix-notification-targeting

Conversation

@HamptonMakes
Copy link
Copy Markdown
Collaborator

New comments now only notify plan authors (created_by_user + author-role collaborators), not all reviewers. Reviewer A leaving feedback no longer pings Reviewers B, C, D who have nothing to do with that thread.

Reviewers are still notified for replies to threads they participated in via the existing thread_participant_ids logic.

New comments now only notify plan authors (created_by_user + author-role
collaborators), not all reviewers. Reviewer A leaving feedback no longer
pings Reviewers B, C, D who have nothing to do with that thread.

Reviewers are still notified for replies to threads they participated in.
@HamptonMakes HamptonMakes merged commit c0bfc8e into main Apr 3, 2026
4 checks passed
@HamptonMakes HamptonMakes deleted the fix-notification-targeting branch April 3, 2026 20:02
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