-
Notifications
You must be signed in to change notification settings - Fork 12k
LazyRoutes Not working any more #3733
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
Comments
Also started getting this last night. Steps to reproduce. ng new / ng serve. delete the node_modules folder, do a npm install. on reinstall I get `| => ng serve ERROR in Cannot read property 'listLazyRoutes' of undefined` |
Yes Yes! it is the most. |
@monoflash Do you have some example code - maybe a a git repository ? |
I can not show all the code. src/app/app-routing.module.ts:
|
@monoflash Try to replace all app/ with ./ there was a change i couldn't find anymore... |
Today project works on [email protected] it's a working code and yes, i try to use version > 19-3 and replace 'app/' to './' |
@monoflash Maybe you can show me the routes for example from SignupModule? |
SignupModule in this link (tell me when you look): Yes, after update angular-cli i upgrade package.json |
hmm, can't see the problem. Sorry Maybe you can build a small example plunker or repository so that we can check the problem. |
Ok, I'll make an example, but a little later. |
If you can get me a simple repro showing the problem I can try and look at it. |
I create test project. It works on [email protected] To my great disappointment, I could not check this test project on v1.0.0-beta.24, after update angular to v1.0.0-beta.24 i "caught" the problem described here: #3803 |
I did a fresh checkout from your example. Both branches works as expected ... I work on Ubuntu 16.04
I deleted the node_modules folder and checked Did you reload all deps after upgrading? Do you have some more details from your environment ? First i thought it wasn't working because in |
Yes I reload all deps after upgrading. $ npm version |
FYI: I upgraded this project to Angular CLI 25.5 and chunking IS working with AOT and lazy loaded routes - https://github.com/rodneyjoyce/lazy-routes |
Lazy loaded routes are working even in AoT in beta.26. Non lazy loaded routes pointing to other modules are still not working in AoT. There are a couple promising Pull Requests for that but anyway, that's a separate issue. This one can be closed. |
Closing as lazy routes are working, and we do test it. There are some bugs that make it not work, mainly #2496, but that is being tracked in that issue. |
This issue has been automatically locked due to inactivity. Read more about our automatic conversation locking policy. This action has been performed automatically by a bot. |
in version [email protected] LazyRoutes works fine
in version v1.0.0-beta.22, v1.0.0-beta.23, v1.0.0-beta.24
LazyRoutes Not working anything in the same code! :(
You can test the project?
I understand that the lack of back compatibility is a contagious disease of modern front-end, take a pill already!
Cry from the heart! Without the hope of understanding...
The text was updated successfully, but these errors were encountered: