Skip to content

[stable34] fix(mail): embed image logo only when using useTemplate - #65427

Open
backportbot[bot] wants to merge 1 commit into
stable34from
backport/65422/stable34
Open

backportbot[bot] wants to merge 1 commit into
stable34from
backport/65422/stable34

Conversation

@backportbot

@backportbot backportbot Bot commented Oct 12, 2026 •

Copy link
Copy Markdown

Backport of #65422

Warning, This backport's changes differ from the original and might be incomplete ⚠️

Todo

  • Review and resolve any conflicts
  • Review and verify the backported changes
  • Amend HEAD commit to remove the line stating to skip CI

Learn more about backports at https://docs.nextcloud.com/server/stable/go.php?to=developer-backports.

The reason for this is that the change that introduced the embedded logo
broke the logo for all apps that did not call useTemplate.

This change uses the logo url by default and replaces it with cid:logo
only when the image is actually attached to the mail.

Signed-off-by: Salvatore Martire <4652631+salmart-dev@users.noreply.github.com>
Assisted-by: ClaudeCode:claude-opus-5-5
@salmart-dev
salmart-dev force-pushed the backport/65422/stable34 branch from b66686e to 74a7817 Compare October 12, 2026 15:58
@salmart-dev
salmart-dev marked this pull request as ready for review October 12, 2026 16:05
@salmart-dev
salmart-dev requested a review from a team as a code owner October 12, 2026 16:05
@salmart-dev
salmart-dev requested review from icewind1991 and sorbaugh and removed request for a team October 12, 2026 16:05
@salmart-dev

Copy link
Copy Markdown
Contributor

Verified manually on stable34

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants