Skip to content

Commit

Permalink
fix: 첫 기록 시도 시, 마지막 시간이 +50분 자동 설정될 때 바텀시트에도 반영되도록 수정
Browse files Browse the repository at this point in the history
  • Loading branch information
wokbjso committed Sep 13, 2024
1 parent d2fb58d commit 3ff19c9
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 14 deletions.
5 changes: 1 addition & 4 deletions features/record/components/organisms/form.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -370,10 +370,7 @@ export function Form({ prevSwimStartTime, prevSwimEndTime }: FormProps) {
defaultTotalLap={data?.data.totalLap}
defaultTotalMeter={data?.data.totalMeter}
/>
<TimeBottomSheet
prevSwimStartTime={prevSwimStartTime}
prevSwimEndTime={prevSwimEndTime}
/>
<TimeBottomSheet startTime={startTime} endTime={endTime} />
</FormProvider>
);
}
Expand Down
17 changes: 7 additions & 10 deletions features/record/components/organisms/time-bottom-sheet.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -20,14 +20,11 @@ import {
} from '../../utils';

interface TimeBottomSheetProps {
prevSwimStartTime?: string;
prevSwimEndTime?: string;
startTime?: string;
endTime?: string;
}

export function TimeBottomSheet({
prevSwimStartTime,
prevSwimEndTime,
}: TimeBottomSheetProps) {
export function TimeBottomSheet({ startTime, endTime }: TimeBottomSheetProps) {
const { getValues, setValue } = useFormContext();
const [timeBottmSheetState, setTimeBottmSheetState] =
useAtom(timeBottomSheetState);
Expand All @@ -39,10 +36,10 @@ export function TimeBottomSheet({
}>(defaultPickerValue);

useEffect(() => {
if (timeBottmSheetState.variant === 'start')
setPickerValue(convertToPickerValue(prevSwimStartTime));
if (timeBottmSheetState.variant === 'end')
setPickerValue(convertToPickerValue(prevSwimEndTime));
if (Boolean(startTime) && timeBottmSheetState.variant === 'start')
setPickerValue(convertToPickerValue(startTime));
if (Boolean(endTime) && timeBottmSheetState.variant === 'end')
setPickerValue(convertToPickerValue(endTime));
// eslint-disable-next-line react-hooks/exhaustive-deps
}, [timeBottmSheetState.variant]);

Expand Down

0 comments on commit 3ff19c9

Please sign in to comment.