Unify namespace storage usage view
What does this MR do and why?
It unifies the storage usage views for both free/paid tiers, any user will now have the same view.
Note: cleaning-up the backend helpers is being handled in !107382 (merged).
Note 2: the MR touches some QA related selectors, but it won't be a problem.
Screenshots or screen recordings
condition | before | after |
---|---|---|
paid namespace | ||
free namespace |
How to set up and validate locally
To see the old design (graph) for free namespaces
- Checkout
master
branch - Enable
namespace_storage_limit
feature flag (bin/rails console
thenFeature.enable(:namespace_storage_limit)
). - Navigate to an owned free namespace
- From left sidebar > Settings > Usage Quotas > Storage
- Observe the storage top section with the graph view
To see the old design (old cards) for paid namespaces
- Checkout
master
branch - Disable
namespace_storage_limit
feature flag (bin/rails console
thenFeature.disable(:namespace_storage_limit)
). - Enable
automatic_purchased_storage_allocation
setting (bin/rails console
then::Gitlab::CurrentSettings.update(automatic_purchased_storage_allocation: true)
) - Navigate to an owned paid namespace
- From left sidebar > Settings > Usage Quotas > Storage
- Observe the storage top section with the old cards view
To see the unified design for any namespace
- Checkout
375651-unify-namespace-storage-limit-view
branch - Disable
namespace_storage_limit
feature flag (bin/rails console
thenFeature.disable(:namespace_storage_limit)
). - Enable
automatic_purchased_storage_allocation
setting (bin/rails console
then::Gitlab::CurrentSettings.update(automatic_purchased_storage_allocation: true)
) - Navigate to an owned free or paid namespace
- From left sidebar > Settings > Usage Quotas > Storage
- Observe the storage top section with the cards view
MR acceptance checklist
This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.
-
I have evaluated the MR acceptance checklist for this MR.
Related to #375651
Edited by Ammar Alakkad