-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
set desplayName
to name
by default
#9719
base: 4.18
Are you sure you want to change the base?
Conversation
@blueorangutan package |
@DaanHoogland a [SL] Jenkins job has been kicked to build packages. It will be bundled with KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress. |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## 4.18 #9719 +/- ##
============================================
- Coverage 12.27% 12.27% -0.01%
+ Complexity 9336 9333 -3
============================================
Files 4699 4699
Lines 414691 414695 +4
Branches 51161 52087 +926
============================================
- Hits 50894 50887 -7
- Misses 357472 357484 +12
+ Partials 6325 6324 -1
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
Packaging result [SF]: ✔️ el8 ✔️ el9 ✔️ debian ✔️ suse15. SL-JID 11176 |
@DaanHoogland the API response does have the "displayname" in place. but the name/displayname is not displayed on UI. as a simple workaround, can we revert #8503 ? @DaanHoogland |
yes, you are right, I'll creat another PR, reapply the changes here, and work on this later. |
@weizhouapache a Jenkins job has been kicked to build UI QA env. I'll keep you posted as I make progress. |
UI build: ✔️ |
b7a8a71
to
7e3bf26
Compare
Description
This PR prevents a null display name on VMs (see #8503 (comment))
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?