Skip to content
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

fix: missing BrowserModule.withServerTransition #2

Merged
merged 1 commit into from
Feb 26, 2017
Merged

fix: missing BrowserModule.withServerTransition #2

merged 1 commit into from
Feb 26, 2017

Conversation

justindujardin
Copy link
Contributor

Without this, angular throws a helpful error while serving the page:

Unhandled Promise rejection: renderModule[Factory]() requires the use of BrowserModule.withServerTransition() to ensure
the server-rendered app can be properly bootstrapped into a client app. ; Zone: <root> ; Task: Promise.then ; Value: { Error: renderModule[Factory]() requires the use of BrowserModule.withServerTransition() to ensure
the server-rendered app can be properly bootstrapped into a client app.
    at new Error (native)
    at Error.v (/Users/coolpath/ng-universal-demo/dist/main.js:251:2215)
    at /Users/coolpath/ng-universal-demo/dist/main.js:155:16564
    at e.invoke (/Users/coolpath/ng-universal-demo/dist/main.js:251:8988)
    at n.run (/Users/coolpath/ng-universal-demo/dist/main.js:251:4723)
    at /Users/coolpath/ng-universal-demo/dist/main.js:251:1787
    at e.invokeTask (/Users/coolpath/ng-universal-demo/dist/main.js:251:9672)
    at n.runTask (/Users/coolpath/ng-universal-demo/dist/main.js:251:5372)
    at o (/Users/coolpath/ng-universal-demo/dist/main.js:251:499)
    at Server.invoke (/Users/coolpath/ng-universal-demo/dist/main.js:251:10588)
    at emitTwo (events.js:106:13)
    at Server.emit (events.js:191:7)
    at HTTPParser.parserOnIncoming [as onIncoming] (_http_server.js:546:12)
    at HTTPParser.parserOnHeadersComplete (_http_common.js:99:23)
...

@robwormald
Copy link
Owner

👍

@robwormald robwormald merged commit 006f0e1 into robwormald:master Feb 26, 2017
@justindujardin justindujardin deleted the patch-1 branch February 26, 2017 16:28
@usernamealreadyis
Copy link

#6

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants