-
Notifications
You must be signed in to change notification settings - Fork 3k
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
Error looking for rxjs/operators.js #2971
Comments
Is this issue fixed? |
We ran into this issue as well, and pinned RxJS to version 5.4.3. This made it work for us again. |
Did the same as @gildebrand but hoping that is just a temporary change and they fix it in 5.5.1 real fast 😬 |
We had to do the same as @gildebrand. |
anyone mind sharing small repo to test out this failure? I'm trying to look into this now. |
I would be willing to bet this is using SystemJS. Internally we shouldn't be referring to |
@jasonaden that's correct that this occurs with SystemJS! |
I have pushed fix, there'll be patch release once it's available. |
Thanks version 5.4.3 worked for me.
From: Anton Gildebrand [mailto:notifications@github.com]
Sent: Wednesday, October 18, 2017 2:47 PM
To: ReactiveX/rxjs <rxjs@noreply.github.com>
Cc: nagarajpatil <nagaraj.patil@hotmail.com>; Comment <comment@noreply.github.com>
Subject: Re: [ReactiveX/rxjs] Error looking for rxjs/operators.js (#2971)
We ran into this issue as well, and pinned RxJS to version 5.4.3. This made it work for us again.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub<#2971 (comment)>, or mute the thread<https://github.com/notifications/unsubscribe-auth/AVopn1BQk0hRRMhSxfs-xWrnK7h_kICZks5stnHngaJpZM4P-Ha->.
|
I've also reverted back to 5.4.3 for now. |
Having the same problem as well. |
I'm using webpack (Angular CLI), and all the operator functions are undefined at run-time for me with 5.5.0. Specifically, I'm getting an error when Is this the same bug described here, or should I report it separately? |
@buu700 I think this is a separate issue. If you could file an issue with a reproduction, that would be great. |
Got it, thanks! Actually, upon further investigation, I'm not sure whether this is an rxjs or Angular bug, so I'll report the details in a ticket there and CC you. Edit: Actually this is probably just a package version mismatch. |
I had this problem too (fixed by reverting the version) and I am using SystemJS. Is only WebPack going to be supported in the future? |
@shaycraft Nope. Just waiting for a |
This the file |
I maintain an automatic parallel package of RxJS, which produces a SystemJS bundle: https://www.npmjs.com/package/rxjs-system-bundle ... and of course I hit the same problem. I worked around it: OasisDigital/rxjs-system-bundle@b0f714a#diff-0b83f9dedf40d7356e5ca147a077acb4R9 with a (hopefully very temporary) hack consisting of a single
I don't recommend this, I just did it to keep producing a working bundle. Hopefully the need will evaporate with the upcoming 5.5.1. |
@kylecordes I'm still seeing this issue with my SystemJS bundle on EDIT: But I suspect that this is a problem with my |
@pe8ter no i am seeing the same thing. They are requiring "operators" which is a folder which isn't allowed. Only webpack (or other bundler) allows import of folder where it assumes you mean the index inside of the folder. There is no way for system.js in the client to do this. This needs to be fixed as it is a breaking change. |
@babeal What you're saying is true of the code as it is arrives from RxJS, but it is possible to work around it, and I have done so with an awful hack that I mentioned a few messages up. My hack creates the missing operators.ts file. It is ugly but it does work fine. When I get a chance I will post a Plunkr demonstrating it. |
@jasonaden ... did we understand wrongly that version 5.5.1 eliminates the problem with missing |
@kylecordes yea thanks. I hear you it's easy to fix but I don't want to have to do that for the 15 projects I maintain so I will lock to 5.4.x until this is fixed. |
@pe8ter @babeal @mlc-mlapis Here are the changes in RxJS between 5.5.0 and 5.5.1; I don't think any of these are relevant to our SystemJS operators.ts problem here unfortunately. So I think the @kwonoj fix must have been about a different 5.5.0 glitch. Here is a plunkr which shows my SystemJS redistribution of RxJS in action (look at config.js). Take a look at app.ts to see the operator import. |
It looks to me like the issue was introduced here: ba2f586#diff-5fe03a9b5bb68d1a34618cf458d64e1c Is anyone working on a pull request for this? |
@kylecordes ... shouldn't be
|
I did this to make it work:
|
@pitAlex That worked great for me on my Angular 5 app. It's nice to have a workaround that doesn't require editing anything in node_modules. You're missing a colon after "rxjs/operators", BTW. |
This issue is fixed in 5.5.3. All workarounds above can be ignored. |
I had the error : "looking for rxjs/operators.js", I don't know how relevant this is, but the solution was not changing version (of rxjs), nor editing the file named "systemjs.config.js", (adding a seperated referal to rxjs/operators, as a post above suggested to do). That systemjs file contains : "'rxjs': 'npm:rxjs',", which is explaining the availability of eg. rxjs/Observable in angular TS SOLUTION for me was : changing line 3 in service file to : "import { catchError, map, tap } from 'rxjs/operators/index';" EXPLAINED : When checking "./node_modules/rxjs" you won't find a file "operators.js", but you'll find a folder "operators". you could refer to the needed operators, or just all available (choosing for index.js) Hopefully this solves and explains for others following same tutorials as I'm doing. |
I am still having the same issue in Angular 6.0.0? does the issue still exists in latest version? |
@ChandraTerli ... ??? it was the issue with RxJS itself and not with Angular. |
Thank you! for the response, I tried your suggestion but got 403 error saying access to is forbidden.
…Sent from my iPhone
On May 7, 2018, at 5:17 PM, Miloš Lapiš ***@***.***> wrote:
@ChandraTerli ... ??? it was the issue with RxJS itself and not with Angular.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.
|
this is what I is in my Package.Json... "rxjs": "6.1.0" and in SystemJs.Config.., |
and I get this error.. |
@ChandraTerli ... but the error is related to http://localhost:41211/node_modules/rxjs/ 403 (Forbidden) Are you importing what when the error appears? |
initial error was rxjs/operators then when I tried to fix it using what I mentioned above it started giving me this error, no not when importing. I am just trying to build a angular6 project and on initial run I see this error(s). |
{ |
/**
|
@ChandraTerli ... it looks that you use SystemJS in the project. Is it an older app? And you want to migrate to Angular 6 now? Then I recommend you to start with a fresh project and with Angular CLI and migrate you previous app step by step. |
Hello!, taken the @pitAlex recommendations, I makes that change on system.config.js and add a similar rxjs config for the error http://localhost:41211/node_modules/rxjs/ 403 (Forbidden), tested with Angular 6.0.3 and rxjs 6.2.0, and it work!!
|
I'm having a similar issue since rxjs 6:
|
@ShinDarth did you find a fix for the error you posted? I am getting the same error with rxjs 6, angular 6. Appreciate your help if you can give me some pointers on how this can be fixed. |
not yet |
@ursseenu71 FYI: I've raised a question here: https://stackoverflow.com/questions/51288263/plunker-error-get-https-unpkg-com-rxjs6-2-1-operators-js-404 I'll post the solution there in case I find one. |
@ShinDarth ... entry point for an operator is |
This is due to the problem loading it with SystemJS described at: ReactiveX/rxjs#2971
**RxJS version: 5.5.0 **
Additional information:
It looks like there is still one place using the old operators file (src/observable/dom/AjaxObservable.ts) line 7: import { map } from '../../operators';
The text was updated successfully, but these errors were encountered: