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

[TECHNICAL] Technical improvements for user quota #4525

Merged
merged 12 commits into from
Jan 10, 2025

Conversation

joragua
Copy link
Collaborator

@joragua joragua commented Dec 11, 2024

Related Issues

App: #4521

  • 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

Test Plan: https://github.com/owncloud/QA/blob/master/Mobile/Android/Executions/Release_4.5/Quota%20Refactor%20(4525).md

@joragua joragua force-pushed the technical/technical_improvements_for_quota branch from 8e7bfe9 to 2f0df52 Compare December 11, 2024 09:38
@joragua joragua self-assigned this Dec 11, 2024
@joragua joragua linked an issue Dec 11, 2024 that may be closed by this pull request
11 tasks
@joragua joragua force-pushed the technical/technical_improvements_for_quota branch from 1f95f15 to 8d5c35f Compare December 11, 2024 10:45
@joragua joragua marked this pull request as ready for review December 11, 2024 12:12
@joragua joragua requested a review from JuancaG05 December 11, 2024 12:12
Copy link
Collaborator

@JuancaG05 JuancaG05 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good @joragua!! Let's do some changes here 👍

@joragua joragua force-pushed the technical/technical_improvements_for_quota branch from 32ed01c to d7f96a2 Compare December 18, 2024 11:56
@joragua joragua requested a review from JuancaG05 December 18, 2024 12:11
@joragua joragua force-pushed the technical/technical_improvements_for_quota branch 2 times, most recently from 1503d9d to 3453331 Compare December 20, 2024 08:42
Copy link
Collaborator

@JuancaG05 JuancaG05 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just one thing that you didn't apply from the previous round of CR and we're done @joragua 👍. Also, you can rebase the branch so that we have it ready for QA

@joragua joragua force-pushed the technical/technical_improvements_for_quota branch from 3453331 to a9576a3 Compare January 8, 2025 11:23
@joragua joragua requested a review from JuancaG05 January 8, 2025 11:47
Copy link
Collaborator

@JuancaG05 JuancaG05 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@joragua joragua force-pushed the technical/technical_improvements_for_quota branch 3 times, most recently from c9e5dc6 to 79d71a2 Compare January 9, 2025 12:50
@jesmrec
Copy link
Collaborator

jesmrec commented Jan 9, 2025

(1) [FIXED]

About this improvement: #4521 (comment)

The color of the progress bar is now red in status Storage nearly full in the drawer.

But, in the same situation, the color of the bar in the accounts manager is still blue. Both bars should have the same color under the same status

Xiaomi Redmi Note 13
Android 14
79d71a27

@joragua joragua force-pushed the technical/technical_improvements_for_quota branch from 9538af8 to dc5ea3c Compare January 10, 2025 07:30
@joragua
Copy link
Collaborator Author

joragua commented Jan 10, 2025

(1) should be done with the last commit @jesmrec

@joragua joragua force-pushed the technical/technical_improvements_for_quota branch from dc5ea3c to 016fc3d Compare January 10, 2025 07:35
@jesmrec
Copy link
Collaborator

jesmrec commented Jan 10, 2025

Approved on my side.

@joragua joragua merged commit 6c9eb65 into master Jan 10, 2025
7 checks passed
@joragua joragua deleted the technical/technical_improvements_for_quota branch January 10, 2025 09:08
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.

[TECHNICAL] Technical improvements for user quota
3 participants