Skip to content

ui: Updated units names#8543

Merged
yadvr merged 3 commits intoapache:4.18from
OlegChuev:fix/update-memory-units
Jun 10, 2024
Merged

ui: Updated units names#8543
yadvr merged 3 commits intoapache:4.18from
OlegChuev:fix/update-memory-units

Conversation

@OlegChuev
Copy link
Contributor

Description

This PR...

Types of changes

  • Breaking change (fix or feature that would cause existing functionality to change)
  • New feature (non-breaking change which adds functionality)
  • Bug fix (non-breaking change which fixes an issue)
  • Enhancement (improves an existing feature and functionality)
  • Cleanup (Code refactoring and cleanup, that may add test cases)
  • build/CI

Feature/Enhancement Scale or Bug Severity

Feature/Enhancement Scale

  • Major
  • Minor

Bug Severity

  • BLOCKER
  • Critical
  • Major
  • Minor
  • Trivial

Screenshots (if appropriate):

How Has This Been Tested?

How did you try to break this feature and the system with this change?

Before the fix
image

After the fix
image

@OlegChuev OlegChuev changed the title [UI] Updated units names ui: Updated units names Jan 21, 2024
@codecov
Copy link

codecov bot commented Jan 22, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 12.23%. Comparing base (88ff77d) to head (4a090a8).
Report is 303 commits behind head on 4.18.

Additional details and impacted files
@@              Coverage Diff              @@
##               4.18    #8543       +/-   ##
=============================================
- Coverage     12.93%   12.23%    -0.71%     
- Complexity     8944     9290      +346     
=============================================
  Files          2715     4698     +1983     
  Lines        256107   414257   +158150     
  Branches      39938    51114    +11176     
=============================================
+ Hits          33139    50702    +17563     
- Misses       218808   357254   +138446     
- Partials       4160     6301     +2141     
Flag Coverage Δ
unittests 12.23% <ø> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@JoaoJandre JoaoJandre added this to the 4.18.2.0 milestone Jan 30, 2024
@JoaoJandre JoaoJandre linked an issue Jan 30, 2024 that may be closed by this pull request
@DaanHoogland
Copy link
Contributor

@OlegChuev any opinion on our comments? we are planning for 4.18.2 and it would be nice to resolve this before then.

@yadvr yadvr requested a review from DaanHoogland June 10, 2024 06:02
@yadvr yadvr modified the milestones: 4.18.3, 4.19.1.0 Jun 10, 2024
@yadvr
Copy link
Member

yadvr commented Jun 10, 2024

@blueorangutan ui

@blueorangutan
Copy link

@rohityadavcloud a Jenkins job has been kicked to build UI QA env. I'll keep you posted as I make progress.

@yadvr yadvr requested review from shwstppr and weizhouapache June 10, 2024 06:13
@blueorangutan
Copy link

UI build: ✔️
Live QA URL: https://qa.cloudstack.cloud/simulator/pr/8543 (QA-JID-370)

@yadvr yadvr merged commit 0b54fbb into apache:4.18 Jun 10, 2024
dhslove pushed a commit to ablecloud-team/ablestack-cloud that referenced this pull request Jun 17, 2024
* Updated units names

* Update ui/public/locales/en.json

* Update ui/public/locales/en.json

---------

Co-authored-by: Rohit Yadav <rohityadav89@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

No open projects
Status: No status

Development

Successfully merging this pull request may close these issues.

usint for min and max memory are different

7 participants