feat(pci-worklow): Refactoring workflow to improve genericity

packages/manager/apps/pci-workflow/src/api/data/region-workflow.ts

@@ -10,7 +10,7 @@ export type TWorkflowExecution = {
   executedAtTime?: string;
 };
 
-export type TRemoteWorkflow = {
+export type TRemoteInstanceBackupWorkflow = {
   backupName: string;
   name: string;
   id: string;
@@ -20,8 +20,8 @@ export type TRemoteWorkflow = {
   distantRegion?: string;
 };
 
-export const getRegionsWorkflows = async (projectId: string, regionName: string) => {
-  const { data } = await v6.get<TRemoteWorkflow[]>(
+export const getInstanceBackupWorkflows = async (projectId: string, regionName: string) => {
+  const { data } = await v6.get<TRemoteInstanceBackupWorkflow[]>(
     `/cloud/project/${projectId}/region/${regionName}/workflow/backup`,
   );
   return data;
@@ -36,13 +36,13 @@ export type TAddWorkflow = {
   distRegionName: string | null;
 };
 
-export const addWorkflow = async (
+export const addInstanceBackupWorkflow = async (
   projectId: string,
   regionName: string,
   instanceId: string,
   type: TAddWorkflow,
 ) => {
-  const { data } = await v6.post<TRemoteWorkflow[]>(
+  const { data } = await v6.post<TRemoteInstanceBackupWorkflow[]>(
     `/cloud/project/${projectId}/region/${regionName}/instance/${instanceId}/automaticBackup`,
     { ...type },
   );

packages/manager/apps/pci-workflow/src/api/hooks/instance/selector/instances.selector.ts

@@ -3,6 +3,7 @@ import { pick } from 'lodash';
 
 import { TProductAvailability, TRegion } from '@ovh-ux/manager-pci-common';
 
+import { TWorkflowSelectedResource } from '@/api/hooks/workflows';
 import { isSnapshotConsumption } from '@/pages/new/utils/is-snapshot-consumption';
 import { TEFlavor } from '@/types/flavor/entity';
 import { TEInstance } from '@/types/instance/entity';
@@ -118,14 +119,14 @@ export type TInstance = Readonly<{
   flavor: TInstanceFlavor;
   status: TInstanceStatus;
   region: TInstanceRegion;
-
   autoBackup: TInstanceAutoBackup;
 }>;
 
 export const buildInstanceId = (id: string, region: string) => ({ id, region }) as TInstance['id'];
-
-export const isSameInstanceId = (a: TInstance['id'], b: TInstance['id']): boolean =>
-  a.id === b.id && a.region === b.region;
+export const buildInstanceSelectedResource = (
+  id: string,
+  region: string,
+): TWorkflowSelectedResource => ({ id, region, label: id });
 
 export const instancesSelector = (
   instances: TEInstance[],

packages/manager/apps/pci-workflow/src/api/hooks/instance/useInstances.ts

@@ -3,18 +3,12 @@ import { useMemo } from 'react';
 import { queryOptions, usePrefetchQuery, useQueries, useQuery } from '@tanstack/react-query';
 
 import { Filter, applyFilters } from '@ovh-ux/manager-core-api';
-import { getProductAvailabilityQuery } from '@ovh-ux/manager-pci-common';
-import { getProjectRegions } from '@ovh-ux/manager-pci-common';
+import { getProductAvailabilityQuery, getProjectRegions } from '@ovh-ux/manager-pci-common';
 import { ColumnSort, PaginationState } from '@ovh-ux/manager-react-components';
 
 import { getFlavor } from '@/api/data/flavor';
 import { getInstances } from '@/api/data/instance';
-import {
-  TInstance,
-  instancesSelector,
-  isSameInstanceId,
-  sortResults,
-} from '@/api/hooks/instance/selector/instances.selector';
+import { instancesSelector, sortResults } from '@/api/hooks/instance/selector/instances.selector';
 import { useRegionTranslation } from '@/api/hooks/region';
 import { useMe } from '@/api/hooks/user';
 import { paginateResults } from '@/helpers';
@@ -124,15 +118,3 @@ export const usePaginatedInstances = (
     ),
   };
 };
-
-export const useInstance = (projectId: string, instanceId: TInstance['id'] | null) => {
-  const { instances } = useAllInstances(projectId);
-
-  return {
-    instance: useMemo(
-      () =>
-        instanceId ? (instances.find((i) => isSameInstanceId(i.id, instanceId)) ?? null) : null,
-      [instances, instanceId],
-    ),
-  };
-};

packages/manager/apps/pci-workflow/src/api/hooks/order/order.tsx

@@ -21,6 +21,15 @@ export type ContinentRegion = Pick<TProductAvailabilityRegion, 'enabled' | 'name
   price: number | null;
 };
 
+export const useGetInstanceSnapshotPricingHook = (enabled: boolean) => {
+  return useMemo(() => {
+    if (enabled) {
+      return useInstanceSnapshotPricing;
+    }
+    return () => ({}) as ReturnType<typeof useInstanceSnapshotPricing>;
+  }, [enabled]);
+};
+
 export const useInstanceSnapshotPricing = (projectId: string, instanceId: TInstance['id']) => {
   const { me } = useMe();
   const { translateMicroRegion, translateContinent } = useRegionTranslation();

packages/manager/apps/pci-workflow/src/api/hooks/resource.ts

@@ -0,0 +1,31 @@
+import { TAddon } from '@ovh-ux/manager-pci-common/dist/types/api/data/catalog';
+
+import { TInstance } from '@/api/hooks/instance/selector/instances.selector';
+import { useInstanceSnapshotPricing } from '@/api/hooks/order/order';
+import { TWorkflowResourceId, WorkflowType } from '@/api/hooks/workflows';
+
+export const isSameResource = (a: TWorkflowResourceId, b: TWorkflowResourceId): boolean =>
+  a.id === b.id && a.region === b.region;
+
+export const getResourcePricing = (
+  workflowType: WorkflowType,
+): ((
+  projectId: string,
+  resourceId: TWorkflowResourceId,
+) => {
+  isPending: boolean;
+  pricing: TAddon['pricings'][0] | null;
+}) => {
+  if (workflowType === WorkflowType.INSTANCE_BACKUP) {
+    return (projectId: string, resourceId: TWorkflowResourceId) =>
+      useInstanceSnapshotPricing(projectId, {
+        id: resourceId.id,
+        region: resourceId.region,
+      } as TInstance['id']);
+  }
+
+  return () => ({
+    pricing: null,
+    isPending: false,
+  });
+};

packages/manager/apps/pci-workflow/src/api/hooks/useExecutions.spec.tsx

@@ -6,10 +6,10 @@ import { wrapper } from '@/wrapperRenders';
 
 import { useWorkflowExecutions } from './useExecutions';
 import * as useWorkflowHook from './workflows';
-import { TWorkflow } from './workflows';
+import { TInstanceBackupWorkflow } from './workflows';
 
 describe('useExecutions tests', () => {
-  vi.spyOn(useWorkflowHook, 'useWorkflows').mockReturnValue({
+  vi.spyOn(useWorkflowHook, 'useInstanceBackupWorkflows').mockReturnValue({
     isPending: false,
     data: [
       {
@@ -32,7 +32,7 @@ describe('useExecutions tests', () => {
             executedAt: '2024-07-17T01:05:24Z',
           },
         ],
-      } as TWorkflow,
+      } as TInstanceBackupWorkflow,
     ],
   });
 

packages/manager/apps/pci-workflow/src/api/hooks/useExecutions.tsx

@@ -10,7 +10,7 @@ import { ColumnSort, PaginationState } from '@ovh-ux/manager-react-components';
 import { paginateResults } from '@/helpers';
 
 import { TWorkflowExecution } from '../data/region-workflow';
-import { useWorkflows } from './workflows';
+import { useInstanceBackupWorkflows } from './workflows';
 
 export const defaultCompareFunction =
   (key: keyof TWorkflowExecution) => (a: TWorkflowExecution, b: TWorkflowExecution) => {
@@ -46,7 +46,7 @@ export const useWorkflowExecutions = (
   const { i18n } = useTranslation('pci-common');
   const userLocale = getDateFnsLocale(i18n.language);
 
-  const { data: workflows, isPending: isWorkflowPending } = useWorkflows(projectId);
+  const { data: workflows, isPending: isWorkflowPending } = useInstanceBackupWorkflows(projectId);
 
   return useMemo(() => {
     let mappedExecution: TWorkflowExecution[] = [];