A-Kasaaian / react-advance-jalaali-datepicker

An advance Jalaali (shamsi, persian) datepicker containing multiple datepicker types such as, range datepicker
MIT License
87 stars 17 forks source link

Error clearing the TextBox(s) value of DateRangepicker #22

Closed khadishi closed 5 years ago

khadishi commented 5 years ago

How to clear the TextBox(s) value of DateRangepicker?

Uncaught Error: Invalid Jalaali year -100721 at jalCal (index.js:95) at d2j (index.js:157) at Object.toJalaali (index.js:28) at toJalaali (index.js:871) at F.push../node_modules/moment-jalaali/index.js.jMoment.fn.jYear (index.js:595) at F.jYYYY (index.js:77) at Object.jYYYYjMMjDD (index.js:238) at F.push../node_modules/moment-jalaali/index.js.jMoment.fn.format (index.js:578) at JDatePicker.componentDidUpdate (index.js:145) at commitLifeCycles (react-dom.development.js:18130) at commitAllLifeCycles (react-dom.development.js:19674) at HTMLUnknownElement.callCallback (react-dom.development.js:147) at HTMLUnknownElement.sentryWrapped (index.js:3848) at Object.invokeGuardedCallbackDev (react-dom.development.js:196) at invokeGuardedCallback (react-dom.development.js:250) at commitRoot (react-dom.development.js:19898) at react-dom.development.js:21446 at Object.unstable_runWithPriority (scheduler.development.js:255) at completeRoot (react-dom.development.js:21445) at performWorkOnRoot (react-dom.development.js:21368) at performWork (react-dom.development.js:21273) at performSyncWork (react-dom.development.js:21247) at interactiveUpdates$1 (react-dom.development.js:21532) at interactiveUpdates (react-dom.development.js:2268) at dispatchInteractiveEvent (react-dom.development.js:5086) at HTMLDocument.sentryWrapped (index.js:3848)

SUGGESTION: https://github.com/A-Kasaaian/react-advance-jalaali-datepicker/blob/master/src/index.js Line 68: preSelected.length > 1 ? moment(preSelected, this.props.format).format("jYYYYjMMjDD") : "",

A-Kasaaian commented 5 years ago

Yea, OK. It'll be fixed as a hot fix.

dophlin commented 5 years ago

@A-Kasaaian, When do you think the hotfix will be released? Is it coming soon?

A-Kasaaian commented 5 years ago

@dophlin published