Skip to content
This repository was archived by the owner on Nov 22, 2024. It is now read-only.

Conversation

@alan-agius4
Copy link
Collaborator

…actory-loaderduringng update`

String based lazy loading syntax is not support with Ivy and hence @nguniversal/module-map-ngfactory-loader is no longer required.

When not removed the application is left in a broken state with the following runtime error

NullInjectorError: R3InjectorError[router_RouterModule -> router_Router -> NgModuleFactoryLoader -> InjectionToken MODULE_MAP -> InjectionToken MODULE_MAP -> InjectionToken MODULE_MAP]:

Fixes: #1272

…actory-loader` during `ng update`

String based lazy loading syntax is not support with Ivy and hence `@nguniversal/module-map-ngfactory-loader` is no longer required.

When not removed the application is left in a broken state with the following runtime error

```
NullInjectorError: R3InjectorError[router_RouterModule -> router_Router -> NgModuleFactoryLoader -> InjectionToken MODULE_MAP -> InjectionToken MODULE_MAP -> InjectionToken MODULE_MAP]:
```

Fixes: #1272
@alan-agius4 alan-agius4 added the target: minor target: minor This PR is targeted for the next minor release label Oct 8, 2019
@vikerman vikerman merged commit 5798f19 into angular:master Oct 8, 2019
@angular-automatic-lock-bot
Copy link

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

This action has been performed automatically by a bot.

@angular-automatic-lock-bot angular-automatic-lock-bot bot locked and limited conversation to collaborators Nov 8, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

target: minor target: minor This PR is targeted for the next minor release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

ng update to 9.0 leaves ModuleMapLoader in app.server.ts

3 participants