Skip to content

Conversation

@SebastianKrupinski
Copy link
Contributor

  • Resolves: # Client Ticket

Summary

  • User quota was being displayed in GIB instead of GB

Before

image1

After

image

Checklist

Signed-off-by: SebastianKrupinski <[email protected]>
@SebastianKrupinski SebastianKrupinski requested a review from a team as a code owner June 2, 2025 19:59
@SebastianKrupinski SebastianKrupinski requested review from nfebe, susnux and szaimen and removed request for a team June 2, 2025 19:59
@SebastianKrupinski SebastianKrupinski self-assigned this Jun 2, 2025
@SebastianKrupinski SebastianKrupinski added the 3. to review Waiting for reviews label Jun 2, 2025
@SebastianKrupinski
Copy link
Contributor Author

/compile

@SebastianKrupinski
Copy link
Contributor Author

/backport to stable31

Signed-off-by: nextcloud-command <[email protected]>
@nextcloud-command nextcloud-command requested a review from a team as a code owner June 2, 2025 20:03
@SebastianKrupinski SebastianKrupinski changed the title fix: new account quota value fix(accounts): new account quota value Jun 2, 2025
@come-nc
Copy link
Contributor

come-nc commented Jun 3, 2025

I thought it was GiB on server side 🤔
What is the frontend sending to the server in the end, "25GB", or a number of bytes?

@SebastianKrupinski
Copy link
Contributor Author

I thought it was GiB on server side 🤔 What is the frontend sending to the server in the end, "25GB", or a number of bytes?

A string "25 GB" the UserRow in the user list does the same thing. I'm guessing this is a work around for 32bit support as an integer input would limit the input to 4GB. Idk I though it was odd we where using a string.

@SebastianKrupinski
Copy link
Contributor Author

@come-nc can I get a second approval. Please

@szaimen szaimen removed their request for review June 3, 2025 13:13
@SebastianKrupinski SebastianKrupinski merged commit 604389c into master Jun 3, 2025
144 of 148 checks passed
@SebastianKrupinski SebastianKrupinski deleted the fix/noid-fix-user-create-quota branch June 3, 2025 14:52
@skjnldsv skjnldsv mentioned this pull request Aug 19, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review Waiting for reviews

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants