Skip to content

Commit 023a665

Browse files
committed
4819 - Fix Deployment dialog tabs in Deployment screen 2
1 parent a8aa31d commit 023a665

3 files changed

Lines changed: 9 additions & 4 deletions

File tree

client/src/pages/automation/project-deployments/components/project-deployment-dialog/ProjectDeploymentDialog.tsx

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -52,6 +52,7 @@ interface ProjectDeploymentDialogProps {
5252
projectDeployment?: ProjectDeployment;
5353
projectDeployments?: ProjectDeployment[];
5454
projectDeploymentsLoading?: boolean;
55+
showTabs?: boolean;
5556
triggerNode?: ReactNode;
5657
}
5758

@@ -63,6 +64,7 @@ const ProjectDeploymentDialog = ({
6364
projectDeployment,
6465
projectDeployments,
6566
projectDeploymentsLoading,
67+
showTabs,
6668
triggerNode,
6769
}: ProjectDeploymentDialogProps) => {
6870
const [activeStepIndex, setActiveStepIndex] = useState(0);
@@ -199,6 +201,7 @@ const ProjectDeploymentDialog = ({
199201
projectDeployments={projectDeployments}
200202
projectDeploymentsLoading={projectDeploymentsLoading}
201203
setValue={setValue}
204+
showTabs={showTabs}
202205
/>
203206
),
204207
name: 'Basic',
@@ -414,7 +417,8 @@ const ProjectDeploymentDialog = ({
414417
disabled={
415418
basicStepTab === 'change-version' &&
416419
!projectDeploymentsLoading &&
417-
(projectDeployments?.length ?? 0) === 0
420+
(projectDeployments?.length ?? 0) === 0 &&
421+
showTabs
418422
}
419423
label="Next"
420424
onClick={handleSubmit(handleNextClick)}

client/src/pages/automation/project-deployments/components/project-deployment-dialog/ProjectDeploymentDialogBasicStep.tsx

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -30,6 +30,7 @@ interface ProjectDialogBasicStepProps {
3030
projectDeployments?: ProjectDeployment[];
3131
projectDeploymentsLoading?: boolean;
3232
setValue: UseFormSetValue<ProjectDeployment>;
33+
showTabs?: boolean;
3334
}
3435

3536
const ProjectDeploymentDialogBasicStep = ({
@@ -41,6 +42,7 @@ const ProjectDeploymentDialogBasicStep = ({
4142
projectDeployments,
4243
projectDeploymentsLoading,
4344
setValue,
45+
showTabs,
4446
}: ProjectDialogBasicStepProps) => {
4547
const [selectedDeploymentId, setSelectedDeploymentId] = useState<string | undefined>();
4648
const [currentProjectId, setCurrentProjectId] = useState(getValues('projectId'));
@@ -100,8 +102,6 @@ const ProjectDeploymentDialogBasicStep = ({
100102
};
101103

102104
const hasDeployments = (projectDeployments?.length ?? 0) > 0;
103-
const shouldShowTabs =
104-
changeProjectVersion || projectDeployment?.id != null || projectDeployment?.projectId != null;
105105

106106
const newDeploymentForm = (
107107
<>
@@ -238,7 +238,7 @@ const ProjectDeploymentDialogBasicStep = ({
238238
</>
239239
);
240240

241-
if (!shouldShowTabs) {
241+
if (!showTabs) {
242242
return <div className="flex flex-col gap-4">{newDeploymentForm}</div>;
243243
}
244244

client/src/pages/automation/projects/components/project-list/ProjectListItem.tsx

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -399,6 +399,7 @@ const ProjectListItem = ({project, projectGitConfiguration, remainingTags}: Proj
399399
}}
400400
projectDeployments={projectDeploymentsQuery.data}
401401
projectDeploymentsLoading={projectDeploymentsQuery.isFetching}
402+
showTabs
402403
triggerNode={
403404
<Button
404405
className="hover:bg-surface-neutral-primary-hover"

0 commit comments

Comments
 (0)