Skip to content

Commit

Permalink
fix(step-generation): fix blowout location from multi-dispense dispos…
Browse files Browse the repository at this point in the history
…al (#17013)

closes RESC-356

This PR fixes a bug where the blowout tip position associated with a multi-dispense disposal was not properly wired up. This resulted in the blowout position to always be at the top of the well which caused users issues.
  • Loading branch information
jerader authored Dec 3, 2024
1 parent 2a4a3c6 commit 3821594
Showing 1 changed file with 7 additions and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -166,7 +166,13 @@ export const moveLiquidFormToArgs = (
'dispense_delay_mmFromBottom'
)
const blowoutLocation =
(fields.blowout_checkbox && fields.blowout_location) || null
(fields.blowout_checkbox && fields.blowout_location) ||
(fields.disposalVolume_checkbox &&
path === 'multiDispense' &&
fields.disposalVolume_volume &&
fields.blowout_location) ||
null

const blowoutOffsetFromTopMm =
blowoutLocation != null
? blowout_z_offset ?? DEFAULT_MM_BLOWOUT_OFFSET_FROM_TOP
Expand Down

0 comments on commit 3821594

Please sign in to comment.