[PR #2405] [MERGED] fix: send organisation member removal email to correct user #2278

Closed
opened 2026-02-26 20:33:12 +03:00 by kerem · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/documenso/documenso/pull/2405
Author: @catalinpit
Created: 1/21/2026
Status: Merged
Merged: 1/28/2026
Merged by: @catalinpit

Base: mainHead: fix/organization-delete-wrong-user-message


📝 Commits (2)

  • 20cfff4 fix: send organisation member removal email to correct user
  • cb99ab2 Merge branch 'main' into fix/organization-delete-wrong-user-message

📊 Changes

1 file changed (+4 additions, -2 deletions)

View changed files

📝 packages/trpc/server/organisation-router/delete-organisation-members.ts (+4 -2)

📄 Description

The email notification for organisation member deletion was incorrectly using the userId of the admin performing the deletion instead of the deleted member's userId. This caused emails to identify the wrong person as having left the organisation.

Checklist

  • I have tested these changes locally and they work as expected.
  • I have added/updated tests that prove the effectiveness of these changes.
  • I have updated the documentation to reflect these changes, if applicable.
  • I have followed the project's coding style guidelines.
  • I have addressed the code review feedback from the previous submission, if applicable.

🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/documenso/documenso/pull/2405 **Author:** [@catalinpit](https://github.com/catalinpit) **Created:** 1/21/2026 **Status:** ✅ Merged **Merged:** 1/28/2026 **Merged by:** [@catalinpit](https://github.com/catalinpit) **Base:** `main` ← **Head:** `fix/organization-delete-wrong-user-message` --- ### 📝 Commits (2) - [`20cfff4`](https://github.com/documenso/documenso/commit/20cfff4043d5b432c58145201b6afe0451a12592) fix: send organisation member removal email to correct user - [`cb99ab2`](https://github.com/documenso/documenso/commit/cb99ab2130e42c3a74c59faea4cc52c22a32524b) Merge branch 'main' into fix/organization-delete-wrong-user-message ### 📊 Changes **1 file changed** (+4 additions, -2 deletions) <details> <summary>View changed files</summary> 📝 `packages/trpc/server/organisation-router/delete-organisation-members.ts` (+4 -2) </details> ### 📄 Description The email notification for organisation member deletion was incorrectly using the userId of the admin performing the deletion instead of the deleted member's userId. This caused emails to identify the wrong person as having left the organisation. ## Checklist - [ ] I have tested these changes locally and they work as expected. - [ ] I have added/updated tests that prove the effectiveness of these changes. - [ ] I have updated the documentation to reflect these changes, if applicable. - [ ] I have followed the project's coding style guidelines. - [ ] I have addressed the code review feedback from the previous submission, if applicable. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
kerem 2026-02-26 20:33:12 +03:00
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
starred/documenso#2278
No description provided.