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

Show storage usage. #6072

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Show storage usage. #6072

wants to merge 4 commits into from

Conversation

mohsenD98
Copy link
Collaborator

Description:
This PR implements a ProgressBar that indicates the amount of storage being used in QFieldCloud.


UI Notes:

  • Usage under 90%: Blue color
  • Usage between 90% and 97.5%: Warning (Yellow) color
  • Usage above 97.5%: Red color with a hyperlink that says: “Upgrade to more storage here”

Upcoming:
The API call will be integrated in future PRs, so this feature will remain hidden until those merges are completed.


Glance:
image

@mohsenD98 mohsenD98 requested a review from nirvn March 8, 2025 16:41
@mohsenD98 mohsenD98 self-assigned this Mar 8, 2025
@qfield-fairy
Copy link
Collaborator

qfield-fairy commented Mar 8, 2025

@nirvn
Copy link
Member

nirvn commented Mar 13, 2025

@suricactus , OK to merge this?

@suricactus
Copy link
Collaborator

Is there a reason to rush with merging this? There is no API that is stabilized yet on the QFC side and also there might be more info we would like to show?

Also, there are still 3 occurances of float/doubles that does not have leading 0, which was never used in the codebase before.

My plan is when we finilize the yearly payments in QFC in the next month or so to open the endpoint that serves this data to the client.

@nirvn
Copy link
Member

nirvn commented Mar 14, 2025

>> Also, there are still 3 occurances of float/doubles that does not have leading 0, which was never used in the codebase before.

That's valid, we need to address this @mohsenD98 .

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

Successfully merging this pull request may close these issues.

4 participants