Skip to content
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

Added support for empty date values #293

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

barbossa
Copy link

@barbossa barbossa commented Jul 3, 2016

Fixed error "Invalid date" #183.
For example, django rest framework return null if value is empty. Without this fix, I always get error "Invalid date"

Fixed error "Invalid date" g00fy-#183. 
For example, django rest framework return null if value is empty. Without this fix, I always get error "Invalid date"
@Dr1Ku
Copy link

Dr1Ku commented Jul 26, 2016

👍, would really like to see this merged!

@tiriana
Copy link

tiriana commented Aug 1, 2016

It also seems to fix #182, at least in my case.

@tiriana
Copy link

tiriana commented Aug 2, 2016

@g00fy- What is the status of this PR? It fixes 1 bug and adds very important feature. I would really like to see it merged and tagged. Can you let me know what are the plans for this PR?

@simonepri
Copy link

I agree with @tiriana and @Dr1Ku we want to see this PR merged

@loopyluke
Copy link

loopyluke commented Nov 28, 2016

You seem to have made the change only in dist - it needs to be made in scripts/datePicker.js.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants