Skip to content

feat: add support for Angular2 Universal - #1165

Closed
asgerjensen wants to merge 4 commits into
PatrickJS:masterfrom
asgerjensen:master
Closed

asgerjensen wants to merge 4 commits into
PatrickJS:masterfrom
asgerjensen:master

Conversation

@asgerjensen

Copy link
Copy Markdown
Contributor
  • What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
    Feature - Adds angular2-universal as build-target.

  • What is the current behavior? (You can also link to an open issue here)
    Not using Universal
    Are there any plans to incorporate Angular Universal? #576

  • What is the new behavior (if this is a feature change)?
    New build-targets (watch:universal, server:universal, build:universal).

  • Other information:
    This is probably best put in a separate branch, as I had to downgrade to angular 2.1.0 in order to get around Fix 2.1.1 _private_ import issues angular/universal#606

Also, I made no effort to minimize code duplication, as i wanted to keep the main-line code intact, and only demonstrate how to set up a universal build of the starter.

@yyyyyyuanfei

Copy link
Copy Markdown

Is this ready to be used for the latest version(2.4.1)?

@joshwiens

Copy link
Copy Markdown
Contributor

There is already a universal starter, adding it here would add unnecessary bloat

@joshwiens joshwiens closed this Dec 29, 2016
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