Skip to content

fix(dav): Catch SAB sync errors during upgrade - #39005

Merged
szaimen merged 1 commit into
masterfrom
fix/dav/upgrade-catch-sab-sync-errors
Jun 27, 2023
Merged

fix(dav): Catch SAB sync errors during upgrade#39005
szaimen merged 1 commit into
masterfrom
fix/dav/upgrade-catch-sab-sync-errors

Conversation

@ChristophWurst

@ChristophWurstChristophWurst commented Jun 26, 2023

Copy link
Copy Markdown
Member

Summary

Allow completion of the upgrade from 26 to 27 even if the instance runs into #38902 or #38873.

TODO

  • Improve code
  • Feedback from an affected instance

Checklist

Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
@ChristophWurst

Copy link
Copy Markdown
MemberAuthor

/backport to stable27

@szaimenszaimen 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.

good idea

@szaimenszaimen added this to the Nextcloud 28 milestone Jun 26, 2023
@ChristophWurstChristophWurst added 3. to review Waiting for reviews and removed 2. developing Work in progress labels Jun 26, 2023
@ChristophWurst
ChristophWurst marked this pull request as ready for review June 26, 2023 16:19
@szaimen
szaimen requested review from a team, ArtificialOwl, icewind1991, miaulalala and nfebe and removed request for a teamJune 26, 2023 16:28
@szaimenszaimen mentioned this pull request Jun 26, 2023
30 tasks
@szaimen
szaimen merged commit 06c99e0 into masterJun 27, 2023
@szaimen
szaimen deleted the fix/dav/upgrade-catch-sab-sync-errors branch June 27, 2023 11:01
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to reviewWaiting for reviewsbugfeature: carddavRelated to CardDAV internalsfeature: davfeature: install and update

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

3 participants

@ChristophWurst@miaulalala@szaimen