Skip to content

Conversation

@joragua
Copy link
Collaborator

@joragua joragua commented Oct 14, 2025

Related Issues

App: #4688

  • Add changelog files for the fixed issues in folder changelog/unreleased. More info here
  • Add feature to Release Notes in ReleaseNotesViewModel.kt creating a new ReleaseNote() with String resources (if required)

QA

#4694 (comment)

Reports:

@joragua joragua self-assigned this Oct 14, 2025
@joragua joragua linked an issue Oct 14, 2025 that may be closed by this pull request
12 tasks
@joragua joragua force-pushed the feature/custom_value_for_space_quota branch from 8c43a4f to 4aaf734 Compare October 14, 2025 09:03
@joragua joragua force-pushed the feature/custom_value_for_space_quota branch from f77c9e1 to 060ecb1 Compare October 15, 2025 08:28
@joragua joragua marked this pull request as ready for review October 15, 2025 08:54
@joragua joragua requested a review from jesmrec October 15, 2025 08:54
@jesmrec
Copy link
Collaborator

jesmrec commented Oct 15, 2025

Good job @joragua. Just a few comments to review

@joragua joragua force-pushed the feature/custom_value_for_space_quota branch from 937ed33 to 1de2c8e Compare October 15, 2025 10:15
@joragua joragua requested a review from jesmrec October 15, 2025 10:33
@jesmrec
Copy link
Collaborator

jesmrec commented Oct 15, 2025

LGTM 👍

@jesmrec
Copy link
Collaborator

jesmrec commented Oct 15, 2025

QA

Test plans templates on create and edit quota have been modified to adapt the new way to handle the quota

Checks:

  1. Create Quota
  1. Edit Quota
  1. Permission
  • Space manager (no platform admin), not allowed
  1. Accessibility
  • Talkback
  • Keyboard navigation

@jesmrec
Copy link
Collaborator

jesmrec commented Oct 15, 2025

(1) [FIXED]

  1. Create an space with a quota value, f. ex, 1 GB
  2. In the Android app, open Edit Space
  3. Switch off the quota and Save

Current:

not modified. the PATCH request sends:

{
    "description": "",
    "name": "test",
    "quota": {
        "total": 1000000
    }
}

Expected:

quota modified to No restricted and sent a 0 in the total field

Xiaomi Redmi Note 13
ec6f9ba2

@jesmrec
Copy link
Collaborator

jesmrec commented Oct 15, 2025

(2) (improvement) [DONE]

The Edit space dialog shows the quota value with one decimal, like 25.0 GB . The decimal value could be hidden for "integer" values and only displayed if it's long. What do you mean?

Xiaomi Redmi Note 13
ec6f9ba2

@jesmrec
Copy link
Collaborator

jesmrec commented Oct 15, 2025

(3) [FIXED]

  1. Set quota to a very small value like 0,00001
  2. Edit space

Current:

Quota displayed in floating point 1.0E-5 GB

Expected:

Quota in decimal format like 0,00001

Xiaomi Redmi Note 13
ec6f9ba2

@joragua
Copy link
Collaborator Author

joragua commented Oct 15, 2025

All reports should be fixed with the new commits, @jesmrec! ✅

@joragua joragua force-pushed the feature/custom_value_for_space_quota branch from c146af2 to 873b884 Compare October 16, 2025 06:54
@jesmrec
Copy link
Collaborator

jesmrec commented Oct 16, 2025

(4) [FIXED]

  1. Click on FAB to create a new space
  2. Switch on the quota

In first instance, quota value is empty. As server sets 1GB as default quota, we could set this value as initial quota

Xiaomi Redmi Note 13
c146af2ff

@jesmrec
Copy link
Collaborator

jesmrec commented Oct 16, 2025

about (4), it works fine in creation but in edition. I mean, creating an space with "No restriction", and editing the space by enabling the quota, shows the field empty.

@joragua joragua force-pushed the feature/custom_value_for_space_quota branch from 5a7a182 to f29c1d6 Compare October 16, 2025 08:46
@jesmrec
Copy link
Collaborator

jesmrec commented Oct 16, 2025

Approved on my side, great job!

@joragua joragua merged commit 99201b0 into master Oct 16, 2025
11 checks passed
@joragua joragua deleted the feature/custom_value_for_space_quota branch October 16, 2025 09:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[FEATURE REQUEST] Custom value in Space quota (creation and edition)

4 participants