Skip to content

Fix peerDependency build issue & change size() to length #1726

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

Closed
wants to merge 2 commits into from

Conversation

tsneed290
Copy link

This fixes the build issue as part of #1664.

@steveyj79
Copy link

looks good to me, hopefully it can be merged and a new build done

@tsneed290
Copy link
Author

I just commented on the original ticket. This doesn't actually work as intended, the date time picker pop up is blank. Will cancel and investigate further.

@tsneed290 tsneed290 closed this Aug 10, 2016
@steveyj79
Copy link

ah okay :(

@tsneed290
Copy link
Author

Ugh, it was my own environmental issues. Going through an upgrade and had some old CSS leaking through that prevented the calendar picker from displaying.

Re-opening, sorry for that.

@tsneed290 tsneed290 reopened this Aug 10, 2016
@steveyj79
Copy link

least it works thats all that matters, good work, not sure how you get it signed off to be deployed

@steveyj79
Copy link

steveyj79 commented Aug 10, 2016

@Eonasdan can this be scheduled for a merge

@tsneed290
Copy link
Author

Hi @Eonasdan, this fixes an issue where jQuery 3.0 was not supported. I just read your contributing guide, please let me know if you wish me to switch this PR to the development branch.

As an aside, what is the release schedule? We depend on this project via NPM and it looks like a release hasn't occurred since 2015.

Regards,
T

@steveyj79
Copy link

steveyj79 commented Aug 12, 2016

@tsneed290 i think the pull request needs to point to development not master

Very important notes

Pull requests to the master branch will be closed. Please submit all pull requests to the development >branch.
Do not include the minified files in your pull request. Don't worry, we'll build them when we cut a >release.
Pull requests that do not include a description (title only) and the following will be closed:
What the change does
A use case (for new features or enhancements)

@tsneed290
Copy link
Author

Thanks @steveyj79, will move it.

@tsneed290 tsneed290 closed this Aug 15, 2016
@tsneed290
Copy link
Author

@steveyj79 it looks like the development branch has the fixes needed to support jQuery 3.

While not desired, I am able to depend on the project in my package.json by referencing the git commit hash:

 "dependencies": {
    "eonasdan-bootstrap-datetimepicker": "git://github.com/Eonasdan/bootstrap-datetimepicker.git#d4971777a957d1d3134b1d90a695a3169111f65f",
  }

@Eonasdan When do you think you'll be releasing 4.17.42 to NPM's registry?

@Eonasdan
Copy link
Owner

dev is now master and updated in npm

@steveyj79
Copy link

Cheers @Eonasdan

@tsneed290
Copy link
Author

Thanks @Eonasdan!!

@jnm2
Copy link

jnm2 commented Aug 18, 2016

@Eonasdan, since this is a hotfix, could you please replace the buggy one currently at NuGet?

@ramin25
Copy link

ramin25 commented Aug 18, 2016

Can this be updated in bower please? bower still shows and pushes version 4.17.37 as the latest which uses input.size.

@virzen
Copy link

virzen commented Aug 19, 2016

Wouldn't creating a 4.17.42 tag solve the issue with bower which @ramin25 mentioned?

@meijdenmedia
Copy link

meijdenmedia commented Aug 27, 2016

@Eonasdan or @nikoskalogridis can one of you guys please create a new tag so we can get the changes via bower?

@elanclarkson
Copy link

elanclarkson commented Sep 20, 2016

@tsneed290

Ugh, it was my own environmental issues. Going through an upgrade and had some old CSS leaking through that prevented the calendar picker from displaying.

This actually appears to be a real issue in jQuery 3.1 that has been reported here #1765.

@osbeug001
Copy link

For bower users who are not aware, there is version 4.17.45 which works.

@ghost ghost locked and limited conversation to collaborators Jun 17, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

9 participants