From 9e61ad37f49aa4eae030a9e7c06d3f77849eb2a1 Mon Sep 17 00:00:00 2001 From: Tom Wilkie Date: Fri, 15 Jan 2016 15:25:41 -0800 Subject: [PATCH] Get Pipes working again. They stopped working because of the change to container node IDs (and rendered nodes IDs for containers). The UI was comparing the IDs, which was never safe. I have just removed that code. This does leave the possibility of us having the pipe control operation take a long time, the user navigate to a different node, and then the terminal pop up, but I think thats better that teaching the UI to understand the format of the node IDs. --- client/app/scripts/actions/app-actions.js | 4 ---- 1 file changed, 4 deletions(-) diff --git a/client/app/scripts/actions/app-actions.js b/client/app/scripts/actions/app-actions.js index 436d9efe3..e8cd058dd 100644 --- a/client/app/scripts/actions/app-actions.js +++ b/client/app/scripts/actions/app-actions.js @@ -1,5 +1,3 @@ -import debug from 'debug'; - import AppDispatcher from '../dispatcher/app-dispatcher'; import ActionTypes from '../constants/action-types'; import { updateRoute } from '../utils/router-utils'; @@ -7,8 +5,6 @@ import { doControlRequest, getNodesDelta, getNodeDetails, getTopologies, deletePipe } from '../utils/web-api-utils'; import AppStore from '../stores/app-store'; -const log = debug('scope:app-actions'); - export function changeTopologyOption(option, value, topologyId) { AppDispatcher.dispatch({ type: ActionTypes.CHANGE_TOPOLOGY_OPTION,