ngx-datatable
ngx-datatable copied to clipboard
Upgrade rxjs to ^7.4.0
re: #2073, #2042
What kind of change does this PR introduce? (check one with "x")
- [ ] Bugfix
- [x] Feature
- [ ] Code style update (formatting, local variables)
- [ ] Refactoring (no functional changes, no api changes)
- [ ] Build related changes
- [ ] CI related changes
- [ ] Other... Please describe:
What is the current behavior? (You can also link to an open issue here)
Since the upgrade to angular 13, installing the package warns that one needs to install ^[email protected]
when one already has [email protected]
, which is what angular 13+ now runs on.
What is the new behavior?
Now that we've upgraded the package to support rxjs@^7.4.0
, Angular 13+ users shouldn't see false positive warnings when installing the package.
Does this PR introduce a breaking change? (check one with "x")
- [ ] Yes
- [x] No
If this PR contains a breaking change, please describe the impact and migration path for existing applications: ...
Other information:
+1
is this going to be merged?
please merge it asap
+1
+1
+1
Please merge
+1. Needed for using ngx-datatables with latest versions of Angular.
![image](https://user-images.githubusercontent.com/86669217/163977132-adbd11d5-153d-4a95-872a-089f92600075.png)
Need this please. Also seems the peer dep is still ng11?
![]()
Need this please. Also seems the peer dep is still ng11?
you can use --force to install it, but test it before going productive. For me it doesn't break.
Please merge
+1
+1
Hi Team,
Please merge this PR ASAP. Lot of people are waiting for this update it seems.
Hi Team,
Please merge this PR ASAP. Lot of people are waiting for this update it seems.
At this point, I honestly believe there's no "Team" behind this anymore. This project seems abandoned by maintainers.
Any update here?
https://www.npmjs.com/package/@boring.devs/ngx-datatable (Forked) stackblitz
Can this be merged?
Please find some time to merge 👍