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.

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

[FEATURE REQUEST] Quota improvements from GraphAPI #4411

Open
8 of 16 tasks
jesmrec opened this issue May 23, 2024 · 10 comments · May be fixed by #4496
Open
8 of 16 tasks

[FEATURE REQUEST] Quota improvements from GraphAPI #4411

jesmrec opened this issue May 23, 2024 · 10 comments · May be fixed by #4496

Comments

@jesmrec
Copy link
Collaborator

jesmrec commented May 23, 2024

That comes from #4401 (comment)

Some improvements we could add, taking advantage of GraphAPI:

  • If quota is exceeded, currently we show No storage usage information available. I'd show the bar full instead. We can also take advantage of the state field that comes in the response:
 "quota": {
                "remaining": 0,
                "state": "exceeded",
                "total": 1000000000,
                "used": 1423007244
            },

playing with colors depending on the status is another option (normal, nearing, exceeded)

  • Quota is updated in some scenarios, that we could spread to those ones in which the value changes locally:

    • Adding a new file
    • Deleting a new file
    • (Maybe all of these cases can be handled updating the quota in the synchronization/refresh operation)

They could trigger a new request to update the values

  • Show spaces' quota as well

  • Showing the quota in manage accounts dialog

  • Checking if quota can be retrieved from other different endpoint, without fetching whole info about all spaces.

TASKS

  • Research (if needed)
  • Create branch feature/quota_improvements
  • Development tasks
    • Add state and total values in quota properties
    • Update drawer when quota is exceeded
    • Update drawer when quota is nearing
    • Update drawer when quota is critical
    • Update quota when a new file is added
    • Update quota when a file is removed
    • Show quota in manage accounts dialog
    • ...
    • Implement unit tests (if needed)
  • Code review and apply changes requested
  • Design test plan
  • QA
  • Merge branch feature/feature_name into master
@jesmrec
Copy link
Collaborator Author

jesmrec commented Sep 11, 2024

Put an eye on https://github.com/owncloud/enterprise/issues/5808

Could be another fix/improvement to do.

@joragua
Copy link
Collaborator

joragua commented Oct 8, 2024

EXCEEDED QUOTA

When the quota is exceeded, the progress bar looks:

Captura de pantalla 2024-10-08 a las 14 05 13 (2)

NEARING QUOTA

I don't know how to display the quota when the state is nearing. Now, we have the total and the used with the progress bar. In my opinion, in this case, we can replace the color by yellow/orange or something like that, but change the string I don't think it's a good idea.

@jesmrec @JuancaG05 Take a look and comment if you have more ideas 🚀

@jesmrec
Copy link
Collaborator Author

jesmrec commented Oct 8, 2024

Adding a (nearing) clarification is a good idea?. Just as an idea, we could have three rows:

1st row: progress bar
2nd row: total / used (if used exist). Just this numerical values
3rd row: message

The message is something like:

if exceeded -> The quota is exceeded
if nearing -> Close to the limit (or similar)
...

if no message, the 3rd row is gone

What do you think?

@joragua
Copy link
Collaborator

joragua commented Oct 8, 2024

Yeah, I agree with you. I'll continue with your idea and I'll update the screenshots when it's "finished" 😄

@joragua joragua linked a pull request Oct 9, 2024 that will close this issue
2 tasks
@joragua
Copy link
Collaborator

joragua commented Oct 9, 2024

Here, an update about quota in drawer:

Exceeded Quota Nearing Quota
Captura de pantalla 2024-10-09 a las 18 09 04 Captura de pantalla 2024-10-10 a las 8 19 42
Critical Quota Normal limited quota 
Captura de pantalla 2024-10-10 a las 9 44 17 Captura de pantalla 2024-10-09 a las 18 15 30

(Updated)

@jesmrec
Copy link
Collaborator Author

jesmrec commented Oct 9, 2024

cool! just an small clarification, in the nearing message: i think Close to the quota limit sounds better

@joragua
Copy link
Collaborator

joragua commented Oct 10, 2024

Hi @tbsbdr! 😄 We have some questions related with this issue. Do you agree with the string below numerical quota: The quota is exceeded, Close to the quota limit, Very close to the quota limit? How can we highlight all these strings to get the user's attention? We consider not to use colors because we could have problems with the branding.

@joragua
Copy link
Collaborator

joragua commented Oct 10, 2024

This is how the manage accounts dialog looks with the quota:

Option 1 Option 2
Captura de pantalla 2024-10-11 a las 12 49 45 Captura de pantalla 2024-10-11 a las 12 59 49
Option 3
Captura de pantalla 2024-10-14 a las 10 23 25

Some comments about the options:

Option 1: You can see the quota in the progress bar and also in the text below it. But I you have some accounts at the same time, the dialog could be very big.

Option 2: The first option without the progress bar.

Option 3: One line with progress bar (If you have limit in the account) and used/total quota.

(It will be updated with more ideas)

@jesmrec
Copy link
Collaborator Author

jesmrec commented Oct 14, 2024

option 3 merges 1 and 2. That's my favorite

@tbsbdr
Copy link

tbsbdr commented Oct 15, 2024

@joragua I observed that people don't know the word "quota", because of that I'd start to rename it to storage if you agree.

My proposal is:

  1. The quota is exceeded --> Storage full
  2. Close to the quota limit --> Storage nearly full
  3. Very close to the quota limit --> Storage very nearly full

Would that be an improvement?

get the user's attention

Hm.. I'd expect a color as a visual hint. If somehow possible and if not "too expensive", let's use a color. Note on branding: The comprehensive branding as we still provide it should be cut back in the future. With this in mind, I'd encourage to cut ties with the past where possible which can mean to not pay much attention to branding. just my 2 cents.

screenshot_003195

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants