Skip to content
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

fix(files): updateAll method #49902

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

fix(files): updateAll method #49902

wants to merge 1 commit into from
+2 −1

Conversation

maxschmi
Copy link

@maxschmi maxschmi commented Dec 17, 2024 •
edited
Loading

Summary

There was a small error in the AppList.vue using this.update, but this is not a method of AppList, but of AppItem

Checklist

@solracsf solracsf added this to the Nextcloud 31 milestone Dec 18, 2024
@solracsf solracsf added the 3. to review Waiting for reviews label Dec 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@skjnldsv skjnldsv skjnldsv approved these changes

@susnux susnux Awaiting requested review from susnux

@Pytal Pytal Awaiting requested review from Pytal

At least 2 approving reviews are required to merge this pull request.

Assignees
No one assigned
Labels
3. to review Waiting for reviews
Projects
None yet
Milestone
Nextcloud 31
Development

Successfully merging this pull request may close these issues.

[Bug]: nxt 30.0.4 The "Update All" button to update all apps does not work properly if I am already logged in.
4 participants
@maxschmi @skjnldsv @solracsf @maxaschmTest