Skip to content

Revert "fix(files): updateAll method" - #50920

Merged
skjnldsv merged 2 commits into
masterfrom
fix/updateall
Feb 20, 2025
Merged

Revert "fix(files): updateAll method"#50920
skjnldsv merged 2 commits into
masterfrom
fix/updateall

Conversation

@skjnldsv

@skjnldsvskjnldsv commented Feb 20, 2025

Copy link
Copy Markdown
Member

This reverts commit fbec19c from #50078

The approach was wrong, I don't know why we didn't check deeper...

  • app.update is the string of the version... 🤦
  • this.update is provided by the AppManagement mixin and is correct

The initial issue from #49866 is different and still needs investigation
(looks like the auth dialog isn't present)

Signed-off-by: skjnldsv <skjnldsv@protonmail.com>
@skjnldsvskjnldsv added this to the Nextcloud 32 milestone Feb 20, 2025
@skjnldsv
skjnldsv requested review from a team, Pytal and susnuxFebruary 20, 2025 13:18
@skjnldsvskjnldsv self-assigned this Feb 20, 2025
@skjnldsv
skjnldsv requested review from artonge and removed request for a teamFebruary 20, 2025 13:18
@AltahrimAltahrim mentioned this pull request Feb 20, 2025
5 tasks
@skjnldsv
skjnldsv disabled auto-merge February 20, 2025 14:19
@skjnldsv
skjnldsv merged commit f5cc31c into masterFeb 20, 2025
@skjnldsv
skjnldsv deleted the fix/updateall branch February 20, 2025 14:19
@skjnldsv

Copy link
Copy Markdown
MemberAuthor

/backport to stable31

@susnuxsusnux moved this from 🏗️ In progress to ☑️ Done in 📁 Files teamMar 14, 2025
@nextcloud-botnextcloud-bot mentioned this pull request Aug 19, 2025
@skjnldsvskjnldsv added this to the Nextcloud 32 milestone Oct 2, 2025
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: ☑️ Done

Development

Successfully merging this pull request may close these issues.

[Bug]: NC31RC4 Update all button not working

3 participants

@skjnldsv@susnux@sorbaugh