chore(viz): formating and reflect suggested PR changes + make route t…

…emplate runnable with jbang
lordrip · Oct 26, 2023 · b3d4a52 · b3d4a52
1 parent 32d40c3
commit b3d4a52
Show file tree

Hide file tree

Showing 5 changed files with 11 additions and 7 deletions.
diff --git a/packages/ui/src/components/Visualization/ContextToolbar/FlowType/NewFlow.tsx b/packages/ui/src/components/Visualization/ContextToolbar/FlowType/NewFlow.tsx
@@ -14,7 +14,6 @@ export const NewFlow: FunctionComponent<PropsWithChildren> = () => {
 
   const checkBeforeAddNewFlow = useCallback(
     (flowType: SourceSchemaType) => {
-      //  console.log('selected new flow', flowType);
       const isSameSourceType = entitiesContext.currentSchemaType === flowType;
 
       if (isSameSourceType) {

diff --git a/packages/ui/src/hooks/entities.ts b/packages/ui/src/hooks/entities.ts
@@ -67,7 +67,7 @@ export const useEntities = (): EntitiesContextResult => {
       currentSchemaType: camelResource?.getType(),
       setCurrentSchemaType: setCurrentSchemaType(),
       visualEntities: camelResource.getVisualEntities(),
-      flowTemplateService: flowTemplateService,
+      flowTemplateService,
       camelResource,
       updateCodeFromEntities,
       eventNotifier,

diff --git a/packages/ui/src/models/visualization/flows/flow-templates-service.ts b/packages/ui/src/models/visualization/flows/flow-templates-service.ts
@@ -12,13 +12,15 @@ export class FlowTemplateService {
         return `apiVersion: camel.apache.org/v1
 kind: Pipe
 metadata:
-  name: webhook-binding
+  name: new-pipe-template
 spec:
   source:
     ref:
       kind: Kamelet
       apiVersion: camel.apache.org/v1
       name: timer-source
+      properties:
+        message: hello
   sink:
     ref:
       kind: Kamelet
@@ -29,9 +31,11 @@ spec:
         return `- route:
     from:
       uri: timer:template
+      parameters:
+        period: "1000"
       steps:
-        - to:
-            uri: log:template`;
+        - log:
+            message: template message`;
       default:
         return '';
     }

diff --git a/packages/ui/src/models/visualization/flows/flows-visibility.ts b/packages/ui/src/models/visualization/flows/flows-visibility.ts
@@ -95,7 +95,7 @@ export class VisualFlowsApi {
   }
 
   toggleFlowVisible(flowId: string, isVisible?: boolean) {
-    this.dispatch({ type: 'toggleFlowVisible', flowId: flowId, isVisible: isVisible });
+    this.dispatch({ type: 'toggleFlowVisible', flowId, isVisible });
   }
 
   showAllFlows() {
@@ -109,6 +109,7 @@ export class VisualFlowsApi {
   setVisibleFlows(flows: string[]) {
     this.dispatch({ type: 'setVisibleFlows', flows: flows });
   }
+
   clearFlows() {
     this.dispatch({ type: 'clearFlows' });
   }

diff --git a/packages/ui/src/providers/visible-flows.provider.tsx b/packages/ui/src/providers/visible-flows.provider.tsx
@@ -20,7 +20,7 @@ export const VisibleFlowsProvider: FunctionComponent<PropsWithChildren> = (props
     const flows: IVisibleFlows = {};
 
     entitiesContext?.visualEntities.forEach((visualEntity) => (flows[visualEntity.id] = visibleFlows[visualEntity.id]));
-    const hiddenAll = Object.values(flows).reduce((acc, current) => acc && !current, true);
+    const hiddenAll = Object.values(flows).every((visible) => !visible);
     if (hiddenAll) {
       flows[entitiesContext!.visualEntities[0].id] = true;
     }