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

pull in the latest datatables fix for triggering isArrayLike issue latest jquery causes #193

Closed
dstull opened this issue Sep 5, 2015 · 12 comments

Comments

@dstull
Copy link

dstull commented Sep 5, 2015

see here for the bug: jquery/jquery#2242

and here for the fix: DataTables/DataTablesSrc@485b259e5c02a

@dstull
Copy link
Author

dstull commented Sep 5, 2015

I've decided to just pull in datatables directly to my vendor/assets - seems to be working so far and is up to date. I'm not sure the value this gem provides, other than making install/maintenance easier. However, if it is not up to date/includes all data tables features, it usefulness diminishes...

@istana
Copy link

istana commented Sep 14, 2015

+1 because datatables cannot be used with new jQuery

@vicvega
Copy link
Contributor

vicvega commented Sep 14, 2015

👍

@cknoxrun
Copy link

It's useful for me as I have it as a gem dependency in an engine. @dstull did you create a PR? If not I will.

@dstull
Copy link
Author

dstull commented Sep 15, 2015

I have not.

@mkhairi
Copy link

mkhairi commented Sep 30, 2015

👍

1 similar comment
@benoitwerner
Copy link

+1

@dometto
Copy link

dometto commented Nov 9, 2015

👍 is anyone already working on a PR? @rweng, would you accept such a PR and cut a new release?

@pwim
Copy link

pwim commented Mar 26, 2016

#204 updates the assets to 1.10.10, which has the fix for this issue. A new gem. 3.4.0, has also been issued. So I think this issue can be closed now.

@bartkamphorst
Copy link

Terrific! @pwim Thanks for letting us know. @TigerWolf Appreciate your efforts; thank you! 👍

@rweng
Copy link
Owner

rweng commented Mar 26, 2016

@dometto sure, I am not actively maintaining this project right now, but pull-requests are always welcome and will be merged in. Might have to ping me once or twice about it, though, sometimes I loose track ;)

@pwin - yep, thank you, I'll close this off then

@rweng rweng closed this as completed Mar 26, 2016
@pwin
Copy link

pwin commented Mar 26, 2016

should have been an acknowledgement to @pwim
 -----Original Message-----From: notifications@github.hscsec.cnSent: Sat, 26 Mar 2016 02:33:59 -0700To: jquery-datatables-rails@noreply.github.hscsec.cnSubject: Re: [jquery-datatables-rails] pull in the latest datatables fix for triggering isArrayLike issue latest jquery causes (#193)@dometto sure, I am not actively maintaining this project right now, but pull-requests are always welcome and will be merged in. Might have to ping me once or twice about it, though, sometimes I loose track ;)

@pwin - yep, thank you, I'll close this off then

—You are receiving this because you were mentioned.Reply to this email directly or view it on GitHub

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

No branches or pull requests