-
Notifications
You must be signed in to change notification settings - Fork 869
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fixed 'selecting the 31st day of previous month' bug #848
base: master
Are you sure you want to change the base?
Conversation
When we will release this one? |
@arqex Please focus this issue |
👍 would love to see this resolved as well |
I would love to see this change merged in and updated. This has been an issue for quite some time, and I would like to see it finally get resolved. |
Until the code is not merged, @AndrewDaneSpackman you can use renderDay function to handle this situation. https://codesandbox.io/s/trusting-allen-xpfthw?file=/src/App.js |
I have attempted to implement this fix and, while the displayed value is now correct, the value set in state is not; it is the previous value. As such, this is not useable as a solution for me. |
@arqex Any updates on this? Apart from this issue, this component does everything else we need it to. Maybe even forgoing the rendering of previous/next months days instead, so they cannot be selected? |
@arqex Any updates on this? |
I have rewritten this workaround to fit my needs, so it might help you as well. It should set the value correctly and preserve time |
I fork the repository and fix this issue. Try https://www.npmjs.com/package/@iftek/react-datetime |
Description
Fixed 'selecting the 31st of previous month' bug
Motivation and Context
Why do you think this pull request should be merged?
Does it solve a problem, in that case what problem?
If these changes fixes an open issue, please provide a link to the issue here
Checklist