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

Patch get valid value undefined #217

Merged
merged 2 commits into from
Mar 8, 2016
Merged

Patch get valid value undefined #217

merged 2 commits into from
Mar 8, 2016

Conversation

holtkamp
Copy link
Collaborator

@holtkamp holtkamp commented Mar 7, 2016

As suggested in #210 (comment)

longbill added a commit that referenced this pull request Mar 8, 2016
@longbill longbill merged commit a36b34c into longbill:master Mar 8, 2016
@holtkamp
Copy link
Collaborator Author

holtkamp commented Mar 8, 2016

Thanks, does this justify a 0.1.1?

@longbill
Copy link
Owner

longbill commented Mar 8, 2016

Hi @holtkamp do you wanna be a manager of this project?

@holtkamp
Copy link
Collaborator Author

holtkamp commented Mar 8, 2016

well, just switched from DatePicker UI to this library for a major project, so I will be spending some time on this. My JavaScript skills are not great however 😄

I did encounter some other 'strange' behaviour for which I intended to file a bug report, I assume you got limited time for this component?

@longbill
Copy link
Owner

longbill commented Mar 8, 2016

yes, I do not have enough time for this project. it will be wonderful if you could help

@holtkamp
Copy link
Collaborator Author

holtkamp commented Mar 8, 2016

ok, will do my best 👍

@holtkamp
Copy link
Collaborator Author

holtkamp commented Mar 8, 2016

@holtkamp holtkamp deleted the patch-getValidValue-undefined branch March 8, 2016 12:53
@719media
Copy link

Thanks for patching this, I was running into this bug. It would be helpful if you could update bower.json version from 0.1.0 to 0.1.1

Thanks!

@holtkamp
Copy link
Collaborator Author

@719media aah, yeah, we should automate that.

Created a new release: https://github.com/longbill/jquery-date-range-picker/releases/tag/0.1.2

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.

3 participants