Merge pull request #56923 from nextcloud/fix/update-all

fix(settings): update all button only updates a single app

Author: AndyScherzinger

apps/settings/src/components/AppList.vue

@@ -351,13 +351,14 @@ export default {
 				})
 		},
 
-		updateAll() {
+		async updateAll() {
 			const limit = pLimit(1)
-			this.apps
+			const updateTasks = this.apps
 				.filter((app) => app.update)
 				.map((app) => limit(() => {
 					this.update(app.id)
 				}))
+			await Promise.all(updateTasks)
 		},
 	},
 }

apps/settings/src/mixins/AppManagement.js

@@ -255,11 +255,11 @@ export default {
 		},
 		update(appId) {
 			if (this.app?.app_api) {
-				this.appApiStore.updateApp(appId)
+				return this.appApiStore.updateApp(appId)
 					.then(() => { rebuildNavigation() })
 					.catch((error) => { showError(error) })
 			} else {
-				this.$store.dispatch('updateApp', { appId })
+				return this.$store.dispatch('updateApp', { appId })
 					.catch((error) => { showError(error) })
 					.then(() => {
 						rebuildNavigation()