Skip to content

fix(update): Correctly trigger update for auto-disabled local apps #53986

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 1 commit into from
Jul 21, 2025
Merged

fix(update): Correctly trigger update for auto-disabled local apps #53986

merged 1 commit into from
Jul 21, 2025

Conversation

@nickvergessen
Copy link
Member

When updating a local app we still need to run updateApp so that repair steps and migrations are correctly executed

Fix #53985

Checklist

@nickvergessen
fix(update): Correctly trigger update for auto-disabled local apps
0ace395 
When updating a local app we still need to run updateApp
so that repair steps and migrations are correctly executed

Signed-off-by: Joas Schilling <coding@schilljs.com>
@nickvergessen nickvergessen added this to the Nextcloud 32 milestone Jul 17, 2025
@nickvergessen nickvergessen self-assigned this Jul 17, 2025
@nickvergessen nickvergessen requested a review from a team as a code owner July 17, 2025 10:19
@nickvergessen nickvergessen requested review from Altahrim, artonge and come-nc and removed request for a team July 17, 2025 10:19
@nickvergessen nickvergessen requested review from a team and mejo- July 17, 2025 10:20
@provokateurin provokateurin enabled auto-merge July 17, 2025 10:32
@provokateurin
Copy link
Member

/backport to stable31

@provokateurin
Copy link
Member

/backport to stable30

@nickvergessen nickvergessen disabled auto-merge July 21, 2025 12:09
@nickvergessen nickvergessen merged commit d9e0216 into master Jul 21, 2025
198 of 203 checks passed
@nickvergessen nickvergessen deleted the bugfix/53985/trigger-update-for-autodisabled-local-apps branch July 21, 2025 12:09
This was referenced Jul 21, 2025
Merged
Merged
@skjnldsv skjnldsv mentioned this pull request Aug 19, 2025
Merged
@skjnldsv skjnldsv modified the milestones: Nextcloud 32, Nextcloud 33 Sep 28, 2025
@skjnldsv skjnldsv added this to the Nextcloud 32 milestone Oct 2, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@susnux susnux susnux approved these changes

@mejo- mejo- mejo- approved these changes

@provokateurin provokateurin provokateurin approved these changes

@Altahrim Altahrim Awaiting requested review from Altahrim Altahrim is a code owner automatically assigned from nextcloud/server-backend

@artonge artonge Awaiting requested review from artonge artonge is a code owner automatically assigned from nextcloud/server-backend

@come-nc come-nc Awaiting requested review from come-nc come-nc is a code owner automatically assigned from nextcloud/server-backend

Assignees

@nickvergessen nickvergessen

Labels

3. to review Waiting for reviews bug feature: install and update

Projects

None yet

Milestone

Nextcloud 32

Development

Successfully merging this pull request may close these issues.

[Bug]: updateApp() not called when upgrading apps incompatible with server version (e.g. live-migration repair steps are ignored)

5 participants

@nickvergessen @provokateurin @susnux @mejo- @skjnldsv