From a7861184151741afd8f752b3913e2f38ecb5ead5 Mon Sep 17 00:00:00 2001 From: mabashian Date: Fri, 24 Jan 2020 17:07:15 -0500 Subject: [PATCH] Removes reference to the node unified job template name in the view modal for now. This component is really just a placeholder and this change fixes an error that is thrown when the node's unified job template is deleted. --- .../Modals/NodeViewModal.jsx | 12 +++--------- .../WorkflowJobTemplateVisualizer/Visualizer.jsx | 4 +--- 2 files changed, 4 insertions(+), 12 deletions(-) diff --git a/awx/ui_next/src/screens/Template/WorkflowJobTemplateVisualizer/Modals/NodeViewModal.jsx b/awx/ui_next/src/screens/Template/WorkflowJobTemplateVisualizer/Modals/NodeViewModal.jsx index d615aad8f4..3ef2f07d3c 100644 --- a/awx/ui_next/src/screens/Template/WorkflowJobTemplateVisualizer/Modals/NodeViewModal.jsx +++ b/awx/ui_next/src/screens/Template/WorkflowJobTemplateVisualizer/Modals/NodeViewModal.jsx @@ -2,23 +2,17 @@ import React from 'react'; import { Modal } from '@patternfly/react-core'; import { withI18n } from '@lingui/react'; import { t } from '@lingui/macro'; -import { func, shape } from 'prop-types'; +import { func } from 'prop-types'; -function NodeViewModal({ i18n, onClose, node }) { +function NodeViewModal({ i18n, onClose }) { return ( - + Coming soon :) ); } NodeViewModal.propTypes = { - node: shape().isRequired, onClose: func.isRequired, }; diff --git a/awx/ui_next/src/screens/Template/WorkflowJobTemplateVisualizer/Visualizer.jsx b/awx/ui_next/src/screens/Template/WorkflowJobTemplateVisualizer/Visualizer.jsx index 2523882194..a7ed441967 100644 --- a/awx/ui_next/src/screens/Template/WorkflowJobTemplateVisualizer/Visualizer.jsx +++ b/awx/ui_next/src/screens/Template/WorkflowJobTemplateVisualizer/Visualizer.jsx @@ -907,9 +907,7 @@ function Visualizer({ history, template, i18n }) { onConfirm={() => deleteAllNodes()} /> )} - {nodeToView && ( - setNodeToView(null)} /> - )} + {nodeToView && setNodeToView(null)} />} ); }