formik-antd icon indicating copy to clipboard operation
formik-antd copied to clipboard

DatePicker format error without keepOffset

Open skykingit opened this issue 3 years ago • 1 comments

When i use the datetime ,i found the value is not the current local time so i see your github code ,i found you transfer date to ISOString by toISOString function, but i can set the props "keepOffset" to prevent the transfer when i set keepOffset true,it did't work so i see you node_modules source code ,you just delete the props "keepOffset" i am so angry, your code is different between github and node_module

....................................................... the github code is below : formik-antd/src/date-picker/index.tsx

export const DatePicker = ({ name, validate, onChange, fast, keepOffset, ...restProps }: DatePickerProps) => ( <Field name={name} validate={validate} fast={fast}> {({ field: { value }, form: { setFieldValue, setFieldTouched }, }: FieldProps) => ( <$DatePicker value={value ? moment(value) : undefined} onChange={(date, dateString) => { setFieldValue(name, date ? date.toISOString(keepOffset) : null) setFieldTouched(name, true, false) onChange && onChange(date, dateString) }} {...restProps} /> )} </Field> )

export default DatePicker


the node_module code is

export const DatePicker = ({ name, validate, onChange, fast, ...restProps }: DatePickerProps) => ( <Field name={name} validate={validate} fast={fast}> {({ field: { value }, form: { setFieldValue, setFieldTouched }, }: FieldProps) => ( <$DatePicker value={value ? moment(value) : undefined} onChange={(date, dateString) => { setFieldValue(name, date ? date.toISOString() : null) setFieldTouched(name, true, false) onChange && onChange(date, dateString) }} {...restProps} /> )} </Field> )

export default DatePicker

skykingit avatar Apr 23 '21 12:04 skykingit

this repos master branch is probably ahead of the version you have installed.

Check https://www.npmjs.com/package/formik-antd for the latest version. 2.1.0-beta.5 might solve your problem

jannikbuschke avatar Aug 26 '21 18:08 jannikbuschke