Skip to content

fix(files): Show correct quota size (file size base 2) - #40787

Merged
skjnldsv merged 2 commits into
stable27from
fix/stable27-wrong-quota-files-app
Oct 6, 2023
Merged

fix(files): Show correct quota size (file size base 2)#40787
skjnldsv merged 2 commits into
stable27from
fix/stable27-wrong-quota-files-app

Conversation

@susnux

@susnuxsusnux commented Oct 5, 2023

Copy link
Copy Markdown
Contributor

Summary

While the whole web ui now again uses the correct file sizes, the files app is using the @nextcloud/files library directly.
For stable27 we are using a broken library version, so we need to adjust the calls to the file size function a bit.

Screenshots

beforeafter
beforeafter

Checklist

@susnuxsusnux added this to the Nextcloud 27.1.3 milestone Oct 5, 2023
@susnux
susnux requested review from a team, nfebe, skjnldsv and szaimen and removed request for a teamOctober 5, 2023 14:23
Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
@susnux
susnuxforce-pushed the fix/stable27-wrong-quota-files-app branch from bf3262a to acdc319CompareOctober 6, 2023 12:30

@szaimenszaimen left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM but didnt test

@susnux
susnux requested a review from artongeOctober 6, 2023 12:33
@skjnldsv
skjnldsv merged commit c2d5a60 into stable27Oct 6, 2023
@skjnldsv
skjnldsv deleted the fix/stable27-wrong-quota-files-app branch October 6, 2023 14:36
@blizzzblizzz mentioned this pull request Oct 9, 2023
@dinosmm

Copy link
Copy Markdown

Is this fix merged in 27.1.3 stable?

@skjnldsv

Copy link
Copy Markdown
Member

image

@dinosmm

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@susnux@dinosmm@skjnldsv@szaimen