Refactor AppUpdateProgress to conditionally display progress for the restart stage. Remove unnecessary detail display for the restart status, enhancing clarity in update management.
All checks were successful
farmcontrol/farmcontrol-ui/pipeline/head This commit looks good

This commit is contained in:
Tom Butcher 2026-06-21 23:42:11 +01:00
parent 991c10d48a
commit 8568a8a8ce

View File

@ -112,7 +112,7 @@ const UpdateStage = ({ stage, status, percent, detail }) => {
const resolvedStatus =
status !== 'error' && resolvedPercent === 100 ? 'complete' : status
const color = getStageColor(resolvedStatus, token)
const showProgress = resolvedStatus === 'active'
const showProgress = resolvedStatus === 'active' && stage !== 'restart'
const StatusIcon =
resolvedStatus === 'complete'
@ -179,8 +179,6 @@ const AppUpdateProgress = ({ progress, update, onClose }) => {
const installDetail = installStatus === 'active' ? message : null
const restartDetail = restartStatus === 'active' ? message : null
return (
<Flex vertical gap='middle'>
<Text>
@ -203,11 +201,7 @@ const AppUpdateProgress = ({ progress, update, onClose }) => {
percent={installPercent}
detail={installDetail}
/>
<UpdateStage
stage='restart'
status={restartStatus}
detail={restartDetail}
/>
<UpdateStage stage='restart' status={restartStatus} />
</Flex>
<Modal