Skip to content

Commit 609eaea

Browse files
committed
Merge pull request #123 from vincent99/master
Fix use existing balancer, hide Services
2 parents 17b67dc + dfd1689 commit 609eaea

4 files changed

Lines changed: 26 additions & 18 deletions

File tree

app/components/page-nav/template.hbs

Lines changed: 12 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -11,12 +11,14 @@
1111
<label>Containers</label>
1212
{{/link-to}}
1313

14-
{{#if hasServices}}
15-
{{#link-to "environments" title="Services" classNames="nav-services"}}
14+
{{!#if hasServices}}
15+
{{!#link-to "environments" title="Services" classNames="nav-services"}}
16+
<!--
1617
<i class="ss-layergroup"></i>
1718
<label>Services</label>
18-
{{/link-to}}
19-
{{/if}}
19+
-->
20+
{{!/link-to}}
21+
{{!/if}}
2022

2123
{{#link-to "balancing" title="Balancing" classNames="nav-balancing"}}
2224
<i class="ss-fork"></i>
@@ -27,3 +29,9 @@
2729
<i class="ss-hdd"></i>
2830
<label>Volumes</label>
2931
{{/link-to}}
32+
33+
{{#link-to "registries" title="Registries" classNames="nav-registries"}}
34+
<i class="ss-bookmark"></i>
35+
<label>Registries</label>
36+
{{/link-to}}
37+

app/loadbalancers/new/controller.js

Lines changed: 10 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -119,19 +119,20 @@ export default Ember.ObjectController.extend(Cattle.NewOrEditMixin, EditLoadBala
119119
{
120120
errors.push('One or more listening ports are required');
121121
}
122-
}
123122

124-
errors.pushObjects(this.get('balancer').validationErrors());
125-
errors.pushObjects(this.get('config').validationErrors());
126-
this.get('listenersArray').forEach((listener) => {
127-
errors.pushObjects(listener.validationErrors());
128-
});
123+
errors.pushObjects(this.get('config').validationErrors());
124+
this.get('listenersArray').forEach((listener) => {
125+
errors.pushObjects(listener.validationErrors());
126+
});
129127

130-
if ( (this.get('listenersArray')||[]).filterProperty('sourcePort',8080).get('length') > 0 )
131-
{
132-
errors.push('Port 8080 cannot currently be used as a source port');
128+
if ( (this.get('listenersArray')||[]).filterProperty('sourcePort',8080).get('length') > 0 )
129+
{
130+
errors.push('Port 8080 cannot currently be used as a source port');
131+
}
133132
}
134133

134+
errors.pushObjects(this.get('balancer').validationErrors());
135+
135136
if ( errors.length )
136137
{
137138
this.set('errors',errors.uniq());

app/styles/layout.scss

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -471,12 +471,12 @@ $nav-7: #444;
471471
&.active { background-color: $nav-4; }
472472
}
473473

474-
.nav-volumes {
475-
border-left-color: $nav-5;
476-
&.active { background-color: $nav-5; }
474+
.nav-registries {
475+
border-left-color: $nav-4;
476+
&.active { background-color: $nav-4; }
477477
}
478478

479-
.nav-registries {
479+
.nav-volumes {
480480
border-left-color: $nav-5;
481481
&.active { background-color: $nav-5; }
482482
}

package.json

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,5 @@
11
{
22
"name": "ui",
3-
"version": "0.16.1",
43
"private": true,
54
"directories": {
65
"doc": "doc",

0 commit comments

Comments
 (0)