Skip to content

fix: preserve customReplyToEmail when hideOrganizerEmail is enabled#27941

Merged
anikdhabal merged 1 commit intomainfrom
custom-replyTo
Feb 13, 2026
Merged

fix: preserve customReplyToEmail when hideOrganizerEmail is enabled#27941
anikdhabal merged 1 commit intomainfrom
custom-replyTo

Conversation

@anikdhabal
Copy link
Copy Markdown
Contributor

@anikdhabal anikdhabal commented Feb 13, 2026

Summary by cubic

Ensures emails keep the customReplyToEmail when hideOrganizerEmail is enabled, so replies go to the intended address without exposing the organizer. Applies the same reply-to logic across reminders, the workflow service, and the header helper.

  • Bug Fixes
    • Use customReplyToEmail when hideOrganizerEmail is true; otherwise fall back to organizer email.
    • Add reply-to only when a value exists.
    • Align logic in scheduleEmailReminders, EmailWorkflowService, and getReplyToHeader.

Written for commit 658aa19. Summary will update on new commits.

@graphite-app graphite-app bot added the core area: core, team members only label Feb 13, 2026
@paragon-review
Copy link
Copy Markdown

Paragon: tests updated

2 updated tests generated for this PR.

Updated Tests

  • getReplyToHeader unit tests — New test file for getReplyToHeader covering: basic reply-to behavior, hideOrganizerEmail + customReplyToEmail preservation (the core fix), excludeOrganizerEmail parameter override, and additionalEmails array/string handling with all combinations of hideOrganizerEmail and customReplyToEmail.
  • getReplyToEmail unit tests — Adding new test to existing file

Accept Changes Open in Paragon

Details

Updated Tests

  • getReplyToHeader unit tests (unit)
  • getReplyToEmail unit tests (unit)

Copy link
Copy Markdown
Contributor

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues found across 3 files

@anikdhabal anikdhabal enabled auto-merge (squash) February 13, 2026 17:21
@github-actions
Copy link
Copy Markdown
Contributor

E2E results are ready!

@anikdhabal anikdhabal merged commit c21281a into main Feb 13, 2026
128 of 131 checks passed
@anikdhabal anikdhabal deleted the custom-replyTo branch February 13, 2026 18:09
emrysal pushed a commit that referenced this pull request Feb 18, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

core area: core, team members only ready-for-e2e size/M

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants