Adds "View Graphs" link to service details [Fixes #146157721]

Transitions user to vms-details state with a defalut viewType specified At the moment, graphs component does not appear to be working correctly, can't handle being the default view, following up with pr to fix
ManageIQ · Jun 19, 2017 · 26fce93 · 26fce93
1 parent f6c2af7
commit 26fce93
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 1 deletion.
diff --git a/client/app/services/service-details/service-details.html b/client/app/services/service-details/service-details.html
@@ -337,6 +337,10 @@ <h2 translate>Resources</h2>
                           <a ng-click="$ctrl.customScope.gotoComputeResource(item)" translate>View
                             Details</a>
                         </li>
+                        <li role="menuitem">
+                          <a ui-sref="vms.details({vmId: item.id, viewType: 'graphsView'})" translate>View
+                            Graphs</a>
+                        </li>
                         <li role="menuitem" ng-if="$ctrl.customScope.permissions.viewSnapshots">
                           <a ui-sref="vms.snapshots({vmId: item.id})" translate>View
                             Snapshots</a>

diff --git a/client/app/services/vm-details/vm-details.component.js b/client/app/services/vm-details/vm-details.component.js
@@ -41,7 +41,7 @@ function ComponentController($stateParams, VmsService, ServicesState, sprintf, l
       availableText: __('Available'),
       notAvailable: __("Not Available"),
       vmDetails: {},
-      viewType: 'detailsView',
+      viewType: $stateParams.viewType || 'graphsView',
       viewSelected: viewSelected,
       instance: {},
       cpuChart: UsageGraphsService.getChartConfig(vm.cpuChartConfigOptions),

diff --git a/client/app/states/vms/details/details.state.js b/client/app/states/vms/details/details.state.js
@@ -7,6 +7,7 @@ function getStates() {
   return {
     'vms.details': {
       url: '/:vmId',
+      params: { viewType: null },
       template: '<vm-details \>',
       title: N_('VM Details'),
     },