Skip to content

[stable5.13] fix(ui): save the attachment of its own message in a thread - #14048

Merged
kesselb merged 1 commit into
stable5.13from
backport/14007/stable5.13
Nov 10, 2026
Merged

kesselb merged 1 commit into
stable5.13from
backport/14007/stable5.13

Conversation

@backportbot

@backportbot backportbot Bot commented Nov 6, 2026 •

Copy link
Copy Markdown

Backport of #14007

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.

@backportbot
backportbot Bot requested a review from kesselb November 6, 2026 18:03
@kesselb
kesselb force-pushed the backport/14007/stable5.13 branch from 7236d52 to 28378ba Compare November 6, 2026 21:14
@kesselb
kesselb marked this pull request as ready for review November 6, 2026 21:45
@kesselb
kesselb enabled auto-merge November 6, 2026 21:46
@kesselb
kesselb force-pushed the backport/14007/stable5.13 branch from 28378ba to 894d07f Compare November 8, 2026 18:52
"Save to Files" on a single attachment used the message id from the route,
which in a thread is the selected message, not necessarily the message the
attachment belongs to. It then saved the attachment with the same id of the
selected message, for example an embedded message instead of a PDF.

Assisted-by: ClaudeCode:claude-opus-5-5
Signed-off-by: Dillard Blom <dillard.blom@ensembia.com>
Signed-off-by: Daniel Kesselberg <3902676+kesselb@users.noreply.github.com>
@ChristophWurst
ChristophWurst force-pushed the backport/14007/stable5.13 branch from 894d07f to d9e470f Compare November 10, 2026 12:10
@ChristophWurst

Copy link
Copy Markdown
Member

Skip ci was in fact not removed before :P

@kesselb
kesselb merged commit 041846a into stable5.13 Nov 10, 2026
33 checks passed
@kesselb
kesselb deleted the backport/14007/stable5.13 branch November 10, 2026 12:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants