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
docs(aio): Upgrade server-side rendering example to Angular V6 #23649
docs(aio): Upgrade server-side rendering example to Angular V6 #23649
Conversation
169fd04
to
df5ebbc
Compare
You can preview 169fd04 at https://pr23649-169fd04.ngbuilds.io/. |
You can preview df5ebbc at https://pr23649-df5ebbc.ngbuilds.io/. |
@vikerman Hi. Igor asked me to reach out to you for review and approval on this. Hoping to merge today. It's a change to universal examples to bring them current for v6. Thanks! |
Looking into this now... |
@@ -51,7 +51,7 @@ app.get('*.*', express.static(join(DIST_FOLDER, 'browser'))); | |||
// #docregion navigation-request | |||
// All regular routes use the Universal engine | |||
app.get('*', (req, res) => { | |||
res.render(join(DIST_FOLDER, 'browser', 'index.html'), { req }); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why are we no longer looking for the index.html as part of the DIST_FOLDER?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The change was based off the universal-starter upgrade to V6. Did fix the change from index.html
to index
df5ebbc
to
d3a9a46
Compare
You can preview d3a9a46 at https://pr23649-d3a9a46.ngbuilds.io/. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Approving. Please ensure sample works with the changes.
d3a9a46
to
7bf224e
Compare
You can preview 7bf224e at https://pr23649-7bf224e.ngbuilds.io/. |
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. |
PR Checklist
Please check if your PR fulfills the following requirements:
PR Type
What kind of change does this PR introduce?
What is the current behavior?
Issue Number: N/A
What is the new behavior?
Does this PR introduce a breaking change?
Other information
Once #23521 lands, I'll rebase this branch