fix label key for gpudisplay - #13272
Conversation
|
@vishesh92 a Jenkins job has been kicked to build UI QA env. I'll keep you posted as I make progress. |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #13272 +/- ##
=========================================
Coverage 18.10% 18.11%
- Complexity 16749 16758 +9
=========================================
Files 6037 6037
Lines 542796 542796
Branches 66456 66456
=========================================
+ Hits 98268 98320 +52
+ Misses 433492 433430 -62
- Partials 11036 11046 +10
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
I fail to see how this is an improvement. I would argue other lebals need an extra . after gpu... |
|
UI build: ✔️ |
We need to define 'label.gpudisplay' to render the label value. I avoided duplicating the key 'label.gpu.display' and renamed it to label.gpudisplay to keep it single-sourced. |
|
@sudo87 |
@weizhouapache The following file doesn't exist in either branch, so it doesn't look like there's any need to change the base: |
|
@sudo87 a Jenkins job has been kicked to build UI QA env. I'll keep you posted as I make progress. |
There was a problem hiding this comment.
🟢 Approval recommended
The change is localized and low risk, with only a minor maintainability suggestion around keeping the old i18n key as an alias for compatibility.
Pull request overview
This PR updates the UI i18n translation key used for the GPU Display compute offering option so the label key matches the gpudisplay parameter naming.
Changes:
- Update the Add Compute Offering form to use
$t('label.gpudisplay')for the GPU Display field label. - Rename the English locale key from
label.gpu.displaytolabel.gpudisplay.
File summaries
| File | Description |
|---|---|
| ui/src/views/offering/AddComputeOffering.vue | Switches the tooltip label translation key for the gpudisplay field. |
| ui/public/locales/en.json | Renames/introduces the translation key for “GPU Display”. |
Review details
- Files reviewed: 2/2 changed files
- Comments generated: 1
- Review effort level: Lite
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
|
UI build: ✔️ |

Description
Types of changes
Feature/Enhancement Scale or Bug Severity
Feature/Enhancement Scale
Bug Severity
Screenshots (if appropriate):
How Has This Been Tested?
How did you try to break this feature and the system with this change?