Skip to content

Commit d4e5622

Browse files
committed
fix: update all button only updates a single app
Signed-off-by: Thomas Lamant <[email protected]>
1 parent 5b92f58 commit d4e5622

File tree

5 files changed

+9
-8
lines changed

5 files changed

+9
-8
lines changed

apps/settings/src/components/AppList.vue

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -351,13 +351,14 @@ export default {
351351
})
352352
},
353353
354-
updateAll() {
354+
async updateAll() {
355355
const limit = pLimit(1)
356-
this.apps
356+
const updateTasks = this.apps
357357
.filter((app) => app.update)
358358
.map((app) => limit(() => {
359359
this.update(app.id)
360360
}))
361+
await Promise.all(updateTasks)
361362
},
362363
},
363364
}

dist/settings-apps-view-4529.js

Lines changed: 2 additions & 2 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

dist/settings-apps-view-4529.js.map

Lines changed: 1 addition & 1 deletion
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

dist/settings-vue-settings-apps-users-management.js

Lines changed: 2 additions & 2 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

dist/settings-vue-settings-apps-users-management.js.map

Lines changed: 1 addition & 1 deletion
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

0 commit comments

Comments
 (0)