-
Notifications
You must be signed in to change notification settings - Fork 285
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: add angular 17 to dependencies #532
base: master
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fine by me
@marjan-georgiev could we please get release for this PR. Currently the library is not working with Angular 17 |
@marjan-georgiev change is still not available in latest release, its not working with Angular 17 |
It's not working with Angular 17, release this PR pls. |
as nobobdy is responding anymore i created a package for ng17 here https://www.npmjs.com/package/@kr0san89/ngx-graph sorry that the readme and so on is still wrong, but i didn't want to invest to much time and i still hope that they will merge contributions from the community again :) or let ppl from the community help with it |
What kind of change does this PR introduce? (check one with "x")
What is the current behavior? (You can also link to an open issue here)
Angular 17 not allowed
What is the new behavior?
Allow Angular 17
Does this PR introduce a breaking change? (check one with "x")
If this PR contains a breaking change, please describe the impact and migration path for existing applications: ...
Other information: