-
Notifications
You must be signed in to change notification settings - Fork 75
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
chore: test and update popover - angular v10 #130
Comments
hello @joejordanbrown is there an estimate date for the release ? |
Are there any errors when using Angular 10 with the popover? From my tests, the latest version works on Angular v10 without any issues. I'm working on a lot of changes internally at the moment that will be released as v4 which will be updated v10 of Angular. |
@joejordanbrown Actually I didnot try it. I got an error while upgrading using ng-upgrade that the popover package is not compatible with angular 10 version. Thats why I didnot proceed with the upgrade as I found this issue and the package is already under upgrade. But I will try it out |
Any news on this? |
@joejordanbrown I didn't notice any issues with the popover working on @angular/core 10.0.14 and @angular/material 10.1.3 |
Working fine on @angular/core 10.1.0 and @angular/material 10.2.0 |
Yes, I am also wondering when will the update be ready? |
@misko009e that's because there aren't any issues with the last version working on Angular v10+. Unless you're having issues? |
@joejordanbrown it appears to work fine yes but had to force the update. If possible, definitely would prefer an update to this just to be sure 😁 |
@misko009e I will try to update it once I find some time. There were a lot of new changes I made locally to improve the popover just haven't got the time at the moment to finish it off. |
In addition to this, testing and releasing it for Angular 11 would probably be a good idea |
We are using the popover extensively with angular 11.2.13 and are not facing any issues. Upgrading angular requires --force though ... |
Angular 12 seems to be working too. As this project seems dead, I'll be forking. |
It looks like it's been copied to https://github.com/ng-matero/extensions |
https://github.com/crh225/popover/tree/angular/upgrades |
I created a PR for @crh225's Angular 13 branch. |
#749 Angular 13 upgrade PR |
Trying to use your fork, but getting the following error.
|
If I fork @crh225 's code and build locally, I get the following error upon start. `` ** Angular Live Development Server is listening on localhost:4300, open your browser on http://localhost:4300/ ** ✖ Failed to compile. |
Test and update popover on Angular v10
The text was updated successfully, but these errors were encountered: