Skip to content

[stable32] fix(Propagator): rollback transaction if it fails - #58131

Closed
backportbot[bot] wants to merge 1 commit into
stable32from
backport/57733/stable32
Closed

[stable32] fix(Propagator): rollback transaction if it fails#58131
backportbot[bot] wants to merge 1 commit into
stable32from
backport/57733/stable32

Conversation

@backportbot

@backportbotbackportbotBot commented Feb 6, 2026

Copy link
Copy Markdown

Backport of #57733

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

Todo

  • Review and resolve any conflicts
  • 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.

@CarlSchwan
CarlSchwanforce-pushed the backport/57733/stable32 branch from 685b611 to 7bbd260CompareFebruary 6, 2026 10:02
@CarlSchwan
CarlSchwan marked this pull request as ready for review February 6, 2026 10:02
@CarlSchwan
CarlSchwan requested a review from a team as a code ownerFebruary 6, 2026 10:02
@CarlSchwan
CarlSchwan requested review from Altahrim, ArtificialOwl, icewind1991 and leftybournes and removed request for a teamFebruary 6, 2026 10:02
Comment threadlib/private/Files/Cache/Propagator.php Outdated

@salmart-devsalmart-dev left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Besides a formatting issue LGTM

Signed-off-by: Carl Schwan <carlschwan@kde.org>
@CarlSchwan
CarlSchwanforce-pushed the backport/57733/stable32 branch from 7bbd260 to 481cd6aCompareFebruary 6, 2026 10:14
@nextcloud-botnextcloud-bot mentioned this pull request Feb 11, 2026
@skjnldsvskjnldsv mentioned this pull request Mar 18, 2026
@skjnldsvskjnldsv mentioned this pull request Mar 26, 2026
13 tasks
@nextcloud-botnextcloud-bot mentioned this pull request Apr 20, 2026
@Altahrim

Copy link
Copy Markdown
Collaborator

The PR now have a conflict

@CarlSchwan

Copy link
Copy Markdown
Member

Already done with cbe4a26

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

Labels

3. to reviewWaiting for reviews

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants

@Altahrim@CarlSchwan@salmart-dev@cristianscheid@AndyScherzinger@joshtrichards@blizzz