-
Notifications
You must be signed in to change notification settings - Fork 72
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
Angular 9 (Ivy) issue #61
Comments
I have the same issue as @jongbonga |
Same issue here. |
I think I'm seeing the same issue but with a different error message:
I was able to fix it by manually re-creating this module...:
I guess it's some issue with the way the libraries built...? |
@DanWebb are you able to run it with ivy enabled after your fix? I guess @nogorilla is not maintaining the library anymore... some issues from 2018 are still unanswered. We are on our own people ✊🏼 |
Sorry @jongbonga spoke too soon the above fails on a prod build in v9, it fooled me because it passed tests. I'm sure it'll be something to do with how the libraries built or exported though. |
I don’t want to get rid of this library since all the others are not satisfying my needs! But also, Can’t rock A9 without Ivy... it doesn’t make sense to me. Any other cc validation as good as this one? that compile with Ivy? Thanks for sharing 🙂 |
I've put some time into investigating and I think the issue is to do with the Anyway, there are lots of other similar occurrences coming up since the v9 release...
I'm out of time and none of the listed solutions have worked so if nobody else has an idea I suppose it'll have to be left for a bit until enough people bring it up for an official solution to be released or an alternative lib is written... |
Yes, for now we can just upgrade angular to 9.x and disable Ivy... |
It would be great to be able to use this with Ivy... for now I've removed this library but will gladly add it back into my project once this is fixed. FWIW I think all that has to happen is that the package needs to be redeployed in a non minified fashion. |
Unfortunately this library does not seem to be maintained anymore.. |
would it make sense to fork it and do the necessary updates? |
So we should leave this library for Ivy? |
will be fixed when this PR gets merged #64 |
Cool, so close issue? |
same issue... fixed? |
The PR is merged, and will be included in upcoming release with some other features |
tx! |
Thanks @thekip |
The text was updated successfully, but these errors were encountered: