Skip to content

[FIX] No message when uploading a file with no quota#4587

Merged
joragua merged 4 commits intomasterfrom
fix/snackbar_when_uploading_with_no_quota
May 28, 2025
Merged

[FIX] No message when uploading a file with no quota#4587
joragua merged 4 commits intomasterfrom
fix/snackbar_when_uploading_with_no_quota

Conversation

@joragua
Copy link
Copy Markdown
Collaborator

@joragua joragua commented May 27, 2025

Related Issues

App: #4582

  • 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

QA checks:

Reports:

@joragua joragua self-assigned this May 27, 2025
@joragua joragua linked an issue May 27, 2025 that may be closed by this pull request
@joragua joragua marked this pull request as ready for review May 27, 2025 16:20
@joragua joragua requested a review from jesmrec May 27, 2025 16:20
@jesmrec
Copy link
Copy Markdown
Collaborator

jesmrec commented May 28, 2025

A couple of comments/questions there, @joragua

@joragua joragua requested a review from jesmrec May 28, 2025 08:32
@jesmrec
Copy link
Copy Markdown
Collaborator

jesmrec commented May 28, 2025

moved to (myself) QA

@jesmrec
Copy link
Copy Markdown
Collaborator

jesmrec commented May 28, 2025

QA checks:

  • Upload shortcut with no quota -> snackbar displayed
  • Upload file with no quota -> snackbar displayed
  • Upload many files with no quota -> snackbar displayed
  • Upload many files with quota for some of them -> snackbar displayed
  • Upload photo from camera with no quota -> snackbar displayed

Some regression...

  • Upload shortcut with quota -> snackbar not displayed, shortcut uploaded
  • Upload file with quota -> snackbar not displayed, file uploaded
  • Upload many files with quota -> snackbar not displayed, files uploaded
  • Upload photo from camera with quota -> snackbar not displayed, photo uploaded

@jesmrec
Copy link
Copy Markdown
Collaborator

jesmrec commented May 28, 2025

(1)

I noticed the following behaviour, i am not sure it can be handled but just in case....

  1. Set an account with a quota that is close to the limit
  2. Upload 100 files (or a number of files for which there is not quota enough)

Files start to upload, some of them are uploaded till the quota is exceeded. Then, uploads fail because no storage. This is totally correct ✅

But, there is no snackbar in this case, i guess because quota was not 0 when the operation started but "in the middle".

My guess is, the only way to add the snackbar afterwards is checking the quota in every individual upload, right? would this be worthy just for only showing an snackbar?

Not a blocker in any case... just something to improve

@jesmrec jesmrec closed this May 28, 2025
@jesmrec jesmrec reopened this May 28, 2025
@joragua
Copy link
Copy Markdown
Collaborator Author

joragua commented May 28, 2025

Right now, the quota is not updated after every individual upload (it is only updated when all files have finished with a successful/failed result). In any case, (1) is a good improvement but it implies more changes in the code... 😄

@jesmrec
Copy link
Copy Markdown
Collaborator

jesmrec commented May 28, 2025

Right now, the quota is not updated after every individual upload (it is only updated when all files have finished with a successful/failed result). In any case, (1) is a good improvement but it implies more changes in the code... 😄

got it! for the moment the new approach is more than enough... now users will have line feedback in case of lack of storage.

From my side it's approved.

@joragua joragua merged commit c82c343 into master May 28, 2025
11 checks passed
@joragua joragua deleted the fix/snackbar_when_uploading_with_no_quota branch May 28, 2025 15: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.

[BUG] No message when uploading a file with no quota

3 participants