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

Add client-side rendering along with server-side. #6

Open
wants to merge 21 commits into
base: master
Choose a base branch
from

Conversation

AntoineViau
Copy link

@AntoineViau AntoineViau commented Feb 28, 2017

This repo is linked by the Angular 4.0.0-rc1 changelog. A lot of people, like myself, will eventually try the demo and spend some time try to understand the errors after doing a npm start.
This PR add client-side rendering (CSR) with angular-cli, along with server-side rendering (SSR). The readme is up-to-date. The main focus is to show one code working for SSR and CSR.

@AntoineViau AntoineViau changed the title Add readme for a quick start and update gitignore Add client-side rendering along with server-side. Feb 28, 2017
@mrjmd
Copy link

mrjmd commented Apr 2, 2017

For people curious about server and client side working together in a Universal project, this looks to be a good example: https://github.com/FrozenPandaz/ng-universal-demo

@MarkPieszak
Copy link

Yes I'd recommend that one as well.
Also universal-starter should be updated soon as well

@ofrebourg
Copy link

People who landed on this page might be interested on this demo version, which contains an API example retrieving data from an external web service. I also added support for LESS files.
https://github.com/ofrebourg/ng-universal-demo

However, note that the use of ExtractTextPlugin in webpack.common.js broke the AOT build in that version. webpack.common.js contains a few commented lines with which AOT builds fine, but I'm not a big fan of that solution.

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.

5 participants