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

Fix 1003 Angular 10 Compatibility #1008

Merged
merged 1 commit into from
Sep 14, 2020

Conversation

jziggas
Copy link
Contributor

@jziggas jziggas commented Aug 28, 2020

No description provided.

@valorkin valorkin merged commit ad945e0 into valor-software:master Sep 14, 2020
@valorkin
Copy link
Member

thanks for helping out

@sguiheux
Copy link

Possible to have a release with the fix ? :)
Thanks a lot

@EmaGht
Copy link

EmaGht commented Oct 23, 2020

After a month from the original request of a release with this fix...

I really need this too. Could you please?
Thanks!

@valorkin
Copy link
Member

This was released a while ago, as far as remember version 6+
@daniloff200 cc

@EmaGht
Copy link

EmaGht commented Oct 23, 2020

@valorkin There is no version 6+ of ng2 dragula on npm. The last version available (2.1.1) still fails on NG10 with this error:

image

@valorkin
Copy link
Member

My bad, sorry. Thought I was responding in ngx bootstrap repo

@EmaGht
Copy link

EmaGht commented Oct 23, 2020

Sure, @valorkin, no problem.
Just a heads up tho, in the readme file it reads "Official Angular wrapper", but we are almost at NG11 and this library still doesn't support NG10 so it doesn't really sound "Official". When i want to switch my company repository to the new NG version should i trust the fact this library gets updated or am i better of branching it and mantaining it?
Thanks again!

@valorkin
Copy link
Member

I can give you a maintenance access if you wish and add to our slack so you could me. Published was automatical on merge to master

@EmaGht
Copy link

EmaGht commented Oct 23, 2020

I'd be happy to help.

Only thing left to understand is why it didn't automatically publish after the merge of this PR

@jziggas
Copy link
Contributor Author

jziggas commented Oct 23, 2020

I think because the build failed due to the demo code needing to be updated.

@EmaGht
Copy link

EmaGht commented Oct 23, 2020

Well, it all makes sense now.

I'm okay with either becoming a maintainer or just waiting for the library to be fixed, what suits you guys best.

@valorkin
Copy link
Member

BUILD ERROR
src/components/dragula.module.ts(10,21): error TS2315: Type 'ModuleWithProviders' is not generic

if this will be fixed, and new version could be released

@reda-alaoui
Copy link
Contributor

reda-alaoui commented Oct 26, 2020

Hello @valorkin,

I fixed it here => #1023

@matharu007
Copy link

@valorkin: can you plz confirm if the fix for the BUILD ERROR is merged and release. I am getting the same error using ng 11

@vamidi vamidi mentioned this pull request Jan 27, 2022
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.

6 participants