minor fix around workflow editor

Author: vivek-devtron

src/components/app/details/appConfig/AppComposeRouter.tsx

@@ -59,6 +59,7 @@ export default function AppComposeRouter({
     isBaseConfigProtected,
     reloadEnvironments,
     configProtectionData,
+    filteredEnvIds
 }: AppComposeRouterProps) {
     const { path } = useRouteMatch()
     const renderJobViewRoutes = (): JSX.Element => {
@@ -190,6 +191,7 @@ export default function AppComposeRouter({
                                 isCDPipeline={isCDPipeline}
                                 respondOnSuccess={respondOnSuccess}
                                 getWorkflows={getWorkflows}
+                                filteredEnvIds={filteredEnvIds}
                             />
                         )}
                     />,

src/components/app/details/appConfig/AppConfig.tsx

@@ -89,7 +89,7 @@ export default function AppConfig({ appName, isJobView, filteredEnvIds }: AppCon
     useEffect(() => {
         Promise.all([
             getAppConfigStatus(+appId, isJobView),
-            getWorkflowList(appId),
+            getWorkflowList(appId, filteredEnvIds),
             getAppOtherEnvironmentMin(appId),
             typeof getConfigProtections === 'function' && !isJobView
                 ? getConfigProtections(Number(appId))
@@ -421,6 +421,7 @@ export default function AppConfig({ appName, isJobView, filteredEnvIds }: AppCon
                             isBaseConfigProtected={state.isBaseConfigProtected}
                             reloadEnvironments={reloadEnvironments}
                             configProtectionData={state.configProtectionData}
+                            filteredEnvIds={filteredEnvIds}
                         />
                     </div>
                 </div>

src/components/app/details/appConfig/appConfig.type.ts

@@ -95,6 +95,7 @@ export interface AppComposeRouterProps {
     isBaseConfigProtected?: boolean
     reloadEnvironments:()=> void
     configProtectionData: any[]
+    filteredEnvIds?: string
 }
 
 export interface EnvironmentOverridesProps {

src/components/app/details/triggerView/workflow.service.ts

@@ -66,7 +66,7 @@ const getInitialWorkflows = (
             )
         })
     } else if (isJobView) {
-        return Promise.all([getWorkflowList(id, filteredEnvIds), getCIConfig(id)]).then(([workflow, ciConfig]) => {
+        return Promise.all([getWorkflowList(id), getCIConfig(id)]).then(([workflow, ciConfig]) => {
             return processWorkflow(
                 workflow.result as WorkflowResult,
                 ciConfig.result as CiPipelineResult,
@@ -77,7 +77,7 @@ const getInitialWorkflows = (
             )
         })
     } else {
-        return Promise.all([getWorkflowList(id), getCIConfig(id), getCDConfig(id), getExternalCIList(id)]).then(
+        return Promise.all([getWorkflowList(id, filteredEnvIds), getCIConfig(id), getCDConfig(id), getExternalCIList(id)]).then(
             ([workflow, ciConfig, cdConfig, externalCIConfig]) => {
                 return processWorkflow(
                     workflow.result as WorkflowResult,

src/components/workflowEditor/types.ts

@@ -42,6 +42,7 @@ export interface WorkflowEditProps
     isJobView?: boolean
     envList?: any[]
     ciPipelines?: any[]
+    filteredEnvIds?: string
 }
 
 export interface AddWorkflowState {
@@ -188,6 +189,6 @@ export interface PipelineContext {
     getPrePostStageInEnv?: (isVirtualEnvironment: boolean, isRunPrePostStageInEnv: boolean) => boolean
     isVirtualEnvironment?: boolean
     globalVariables: {
-        stageType?: string, label: string; value: string; format: string; description?: string; variableType?: string 
+        stageType?: string, label: string; value: string; format: string; description?: string; variableType?: string
 }[]
 }

src/components/workflowEditor/workflowEditor.tsx

@@ -79,6 +79,14 @@ class WorkflowEdit extends Component<WorkflowEditProps, WorkflowEditState> {
         }
     }
 
+    componentDidUpdate(prevProps) {
+        if (
+            prevProps.filteredEnvIds !== this.props.filteredEnvIds
+        ) {
+            this.getWorkflows()
+        }
+    }
+
     removeTakeMeThereClickedItem = () => {
         if (typeof Storage !== 'undefined' && localStorage.getItem('takeMeThereClicked')) {
             localStorage.removeItem('takeMeThereClicked')
@@ -94,6 +102,7 @@ class WorkflowEdit extends Component<WorkflowEditProps, WorkflowEditState> {
         getCreateWorkflows(
             this.props.match.params.appId,
             this.props.isJobView,
+            this.props.filteredEnvIds
         )
             .then((result) => {
                 const allCINodeMap = new Map()