-
Notifications
You must be signed in to change notification settings - Fork 42
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
Support for RxJS 6, Angular 6 #72
Comments
Addon from the user @ChandraTerli |
I bypassed this issue using rxjs-compat for now, which is supposed to provide migration until RxJS 7. |
Cool. You are have the solution and I have more time to put this enhance in my schedule. |
rxjs-compat is not working for me, unfortunately. I get " Fatal Error this.subjExternalValue.map is not a function".. using rxjs-compat 6.2.0 |
I get the same error as @rlefever68, but only when I run in production mode. In dev mode, I don't get any errors with rxjs-compat 6.2.0 |
Today I upgraded my general project and I don't note any issues between the ngx-select-ex and Angular 6. Looks like the issue is not actual. Closed. If it is still actual let me know how to reproduce it. |
The issue is still valid. I have v3.5.9 installed and the imports do not comply with rxjs v6. rxjs-compat is supposed to be a stepping stone before completely moving to rxjs v6, and it will not work on rxjs v7. You can check more details about the migration here: https://github.com/ReactiveX/rxjs/blob/master/docs_app/content/guide/v6/migration.md If needed I can issue a PR with the required changes. |
Did you try running in production mode? I still get errors as of yesterday
in production mode.
…On Wed, Jun 20, 2018 at 8:14 AM Mike Drakoulelis ***@***.***> wrote:
The issue is still valid. I have v3.5.9 installed and the imports do not
comply with rxjs v6.
rxjs-compat is supposed to be a stepping stone before completely moving to
rxjs v6, and it will not work on rxjs v7. You can check more details about
the migration here:
https://github.com/ReactiveX/rxjs/blob/master/docs_app/content/guide/v6/migration.md
If needed I can issue a PR with the required changes.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<#72 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/ATjYPkHSUn_YvGQl9Ycs7sp8_EaMYK8aks5t-jyXgaJpZM4T4Ant>
.
|
I build so But. The component is still on Angular 5. Little late I'll upgrade it for Angular 6. Currently, I do not see the issue. |
For a quick adjustment (although this is an easy transition) you can list rxjs-compat in the |
@optimistex , I'm using webpack, not ngbuild, and webpack in production mode gives me the error I mentioned earlier. I created a test project using ng build and noticed I don't see the errors. I will track down the exact webpack configuration issue causing this, but we will still need a code change so that we can eventually remove the rxjs-compat library and have proper rxjs 6 support. Thanks for testing it. |
Just in case anyone is trying to get this working with Webpack 4, you need to disable the sideEffects optimization in production mode.
|
Maybe any ideas, how to resolve it? to this? |
rxjs compatibility worked for me... But is gonna be deprecated on rxjs 7... |
Can you let folks know when it is ready for rxjs 6, without the rxjs-compat module? Thank you. |
I'm using rxjs-compat module but i have this error in console ERROR TypeError: rxjs_Observable__WEBPACK_IMPORTED_MODULE_5__.Observable.from is not a function |
@optimistex the solution from @galvakojis fixes the compilation issue when rxjs-compat isn't used.
Is there any chance you can push this small update so everybody can start use your library without rxjs-compat? |
@ravenwill
|
This solved our problems with the error |
Please fix this. I want to get rid of rxjs-compat. Otherwise, I'm forced to switch to ng-select: |
Doesn't work for me. |
It works for me. Thanks |
The component has upgraded in the 3.6.1 release to the latest Angular: https://github.com/optimistex/ngx-select-ex/releases/tag/3.6.1 |
you are still using compat package, which you should not, if code is fully migrated to rxjs6. this causes:
Could you remove this dependency and migrate code to RxJS6 ? |
I have tried to create pull request (I don't known if it is correct to use fork and pull request). valor-software#971 |
@prikryl Fixed in the development version. Soon will be in a release...
You have to make a pull request into my fork. |
Currently it is not possible to use this with Angular 6 since the rxjs operators follow the old 'rxjs/add/operator/' syntax. It has been deprecated and in RxJS 6 it is now removed, making the migration impossible.
Any chance to update imports? This should be trivial, but might drop some Angular 4 support.
The text was updated successfully, but these errors were encountered: