-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
[AMBARI-22826] Mpack related changes to Add Hosts Page (ishanbha) #178
[AMBARI-22826] Mpack related changes to Add Hosts Page (ishanbha) #178
Conversation
Refer to this link for build results (access rights to CI server needed): |
this.get('inputtedAgainHostNames').push(hostNameArr[i]); | ||
var tempArr = []; | ||
if (this.get('manualInstall')) { | ||
this.get('manuallyInstalledHosts').forEach ( function (host) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You can use mapProperty('hostName')
if(arr.length <= 1) { | ||
return arr.toString(); | ||
} | ||
var formattedText = ""; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You can use arr.join('\n')
if (filter) { | ||
filter = filter.toUpperCase(); | ||
} | ||
this.get('controller.manuallyInstalledHosts').forEach(function (host) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We are trying to minimize directly changing DOM nodes, use class binding instead.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@atkach But here I need to dynamically set display to none or block based on what is being typed in the filter box for each item in the list. I thought fetching by id is quick. How do I bind each item in the list to a class?
What changes were proposed in this pull request?
Changes to be made for the Add Hosts Page including changes to the manual registration of hosts, registration using ssh and table display for the pre-registered hosts.
How was this patch tested?
Previous tests were modified/removed. New tests were added. All unit tests pass.
20392 passing (21s)
125 pending
Please review Ambari Contributing Guide before opening a pull request.
@atkach @hiveww @Jetly-Jaimin @vivekratnavel