fix: creating new release allows for scheduled time to be set

sanity-io · Jan 17, 2025 · 357eb9a · 357eb9a
1 parent bc92a49
commit 357eb9a
Show file tree

Hide file tree

Showing 2 changed files with 69 additions and 21 deletions.
diff --git a/packages/sanity/src/core/releases/tool/components/ScheduledTimePicker.tsx b/packages/sanity/src/core/releases/tool/components/ScheduledTimePicker.tsx
@@ -0,0 +1,49 @@
+import {format, isValid} from 'date-fns'
+import {useCallback, useState} from 'react'
+
+import {MONTH_PICKER_VARIANT} from '../../../../ui-components/inputs/DateInputs/calendar/Calendar'
+import {DatePicker} from '../../../../ui-components/inputs/DateInputs/DatePicker'
+import {LazyTextInput} from '../../../../ui-components/inputs/DateInputs/LazyTextInput'
+
+type ScheduledTimePickerProps = {
+  initialValue?: Date
+  onChange: (date: string) => void
+}
+
+export const ScheduledTimePicker: React.FC<ScheduledTimePickerProps> = ({
+  initialValue,
+  onChange,
+}) => {
+  const [inputValue, setInputValue] = useState<Date | undefined>(initialValue)
+
+  const handleInputChange = useCallback((event: React.FocusEvent<HTMLInputElement>) => {
+    const parsedDate = new Date(event.currentTarget.value)
+
+    if (isValid(parsedDate)) {
+      setInputValue(parsedDate)
+
+      onChange(parsedDate.toISOString())
+    }
+  }, [])
+
+  return (
+    <>
+      <LazyTextInput
+        value={inputValue ? format(inputValue, 'PPp') : undefined}
+        onChange={handleInputChange}
+        readOnly
+      />
+
+      <DatePicker
+        ref={datePickerRef}
+        monthPickerVariant={MONTH_PICKER_VARIANT.carousel}
+        calendarLabels={calendarLabels}
+        selectTime
+        padding={0}
+        value={inputValue}
+        onChange={handleBundlePublishAtChange}
+        showTimezone
+      />
+    </>
+  )
+}
diff --git a/packages/sanity/src/core/releases/tool/detail/ReleaseTypePicker.tsx b/packages/sanity/src/core/releases/tool/detail/ReleaseTypePicker.tsx
@@ -5,10 +5,7 @@ import {isEqual} from 'lodash'
 import {useCallback, useEffect, useMemo, useRef, useState} from 'react'
 
 import {Button, Popover, Tab} from '../../../../ui-components'
-import {MONTH_PICKER_VARIANT} from '../../../../ui-components/inputs/DateInputs/calendar/Calendar'
 import {type CalendarLabels} from '../../../../ui-components/inputs/DateInputs/calendar/types'
-import {DatePicker} from '../../../../ui-components/inputs/DateInputs/DatePicker'
-import {LazyTextInput} from '../../../../ui-components/inputs/DateInputs/LazyTextInput'
 import {getCalendarLabels} from '../../../form/inputs/DateInputs/utils'
 import {useTranslation} from '../../../i18n/hooks/useTranslation'
 import useTimeZone from '../../../scheduledPublishing/hooks/useTimeZone'
@@ -18,6 +15,7 @@ import {type ReleaseDocument, type ReleaseType} from '../../store'
 import {useReleaseOperations} from '../../store/useReleaseOperations'
 import {getReleaseTone} from '../../util/getReleaseTone'
 import {getPublishDateFromRelease, isReleaseScheduledOrScheduling} from '../../util/util'
+import {ScheduledTimePicker} from '../components/ScheduledTimePicker'
 
 export function ReleaseTypePicker(props: {release: ReleaseDocument}): React.JSX.Element {
   const {release} = props
@@ -179,24 +177,25 @@ export function ReleaseTypePicker(props: {release: ReleaseDocument}): React.JSX.
           />
         </TabList>
         {dateInputOpen && (
-          <>
-            <LazyTextInput
-              value={inputValue ? format(inputValue, 'PPp') : undefined}
-              onChange={handleInputChange}
-              readOnly
-            />
-
-            <DatePicker
-              ref={datePickerRef}
-              monthPickerVariant={MONTH_PICKER_VARIANT.carousel}
-              calendarLabels={calendarLabels}
-              selectTime
-              padding={0}
-              value={inputValue}
-              onChange={handleBundlePublishAtChange}
-              showTimezone
-            />
-          </>
+          <ScheduledTimePicker initialValue={inputValue} />
+          // <>
+          //   <LazyTextInput
+          //     value={inputValue ? format(inputValue, 'PPp') : undefined}
+          //     onChange={handleInputChange}
+          //     readOnly
+          //   />
+
+          //   <DatePicker
+          //     ref={datePickerRef}
+          //     monthPickerVariant={MONTH_PICKER_VARIANT.carousel}
+          //     calendarLabels={calendarLabels}
+          //     selectTime
+          //     padding={0}
+          //     value={inputValue}
+          //     onChange={handleBundlePublishAtChange}
+          //     showTimezone
+          //   />
+          // </>
         )}
       </Stack>
     )