luxolus
/
open-consul
2
0
Fork 0
Code Issues 1 Pull Requests Packages Releases Wiki Activity

UI cleanup follow up from #3245. (#3251) 

* Removes unnecessary set for model component which will be null.

* Returns a 404 for a missing node, not a 200 with an empty response.

* Updates built-in web assets.
This commit is contained in:
James Phillips 2017-07-10 09:40:00 -07:00 committed by GitHub
parent 8a52ae9d03
commit 219fb6dd70
3 changed files with 36 additions and 38 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

64
agent/bindata_assetfs.go
View File

File diff suppressed because one or more lines are too long

4
agent/ui_endpoint.go
View File

@ -96,7 +96,11 @@ RPC:
info.Checks = make([]*structs.HealthCheck, 0) info.Checks = make([]*structs.HealthCheck, 0)
} }
return info, nil return info, nil
} else {
resp.WriteHeader(http.StatusNotFound)
return nil, nil
} }
return nil, nil return nil, nil
} }

6
ui/javascripts/app/routes.js
View File

@ -356,12 +356,6 @@ App.NodesShowRoute = App.BaseRoute.extend({
controller.set('content', models.node); controller.set('content', models.node);
controller.set('sessions', models.sessions); controller.set('sessions', models.sessions);
controller.set('tomography', models.tomography); controller.set('tomography', models.tomography);
//
// Since we have 2 column layout, we need to also display the
// list of nodes on the left. Hence setting the attribute
// {{nodes}} on the controller.
//
controller.set('nodes', models.nodes);
} }
}); });