-
Notifications
You must be signed in to change notification settings - Fork 51
Add support for angular 13 #59
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
Comments
@rbalet ok I will check |
Hello, Regards. |
@1andy11 The compiled component version from NPM repo is working as intended in the angular v 13. I have created test project here: Also I have created local blank angular app with the 13 version and also did not see any issues. If you would like I can send you sources which I have used to test the component on the Angular v 13 |
@AlexMiniApps I can do the migration if you're willing to merging it. Should I start with it? Btw, I'm confirming using angular 13 with ionic and it work smoothly |
@AlexMiniApps This is much more a nice to have than a need for now. But not having ivy enable is a problem for the overall optimization of the code & will be a problem in the not to distant future. So for the small effort it cost, this would be a huge step forward. Thx for you work! |
@rbalet I understand you. ok |
@AlexMiniApps The problem was in my configuration of the project. It is strange but specifying "paths" in tsconfig.json break it. When I removed all paths it was working. Thanks. |
@AlexMiniApps I think you can close this issue ? |
@rbalet nope :) I am working on the release with the enabled ivy, but with the backward compatibility of the prev angular versions. I will close the issue by myself. |
@AlexMiniApps I think you should put that much effort on backward compatibility. You just mention on the README and it should be enough. My reasoning being that
|
@rbalet You're right that Angular seems does not have the backward compatibility. I'm thinking about to split the npm package into two versions of the lib - with and without ivy support respectively. |
@AlexMiniApps As you think it good. But people whom are using it in ionic will have to update to the latest angular, if they wish it to be working on latest smartphone os. So splitting it won't be required for ionic users. For angular one, I do think they won't update your library even if you add new feature since they didn't took the time to migrate to ivy (which is already 1 Years old). Let me know if I can be of any help :) |
@AlexMiniApps any update on this, Angular 16 removed |
@wassim-k I will check it. |
Any updates for angular 16 verson? |
Yep. I am working on this, when have a free time. |
@rbalet Thank you for the PR. I am thinking about how to keep non ivy version and ivy together (maybe in the separate packages). I do not want to lose the support of the older Angular versions. |
@AlexMiniApps Fact is, if you wish to keep the IVY version, you wont have the right to go with angular 15 and higher. (Since the compilation change). so people will slowly but surely drop your library for a more modern one. But I would say, you have the following two options 1. Split librariesCreate a second ngx-angular-code-input library that deal with the latest version of angular. 2. Populate a READMEWhat most libraries are doing, This would be an example
Let me know what you wish to do, I'm ready to help you on both case |
@rbalet Thank you for the advice. I am thinking about the splitting. I am planning to keep the consolidate code base and two build systems for different Angular versions. |
Breaking changes: - Enabled Ivy in the compiled library - The minimum supported version of Angular is set to 16 Resolved the issue: - Add support for angular 13 #59
Hello, guys I have published the package with the latest version of 2.0.0. @rbalet Thank you for help. |
Would be nice to have it support angular 13.
cheers
The text was updated successfully, but these errors were encountered: