Skip to content

docs: Update Universal Story to include module map - #7739

Closed
RicardoVaranda wants to merge 1 commit into
angular:masterfrom
RicardoVaranda:patch-1
Closed

docs: Update Universal Story to include module map#7739
RicardoVaranda wants to merge 1 commit into
angular:masterfrom
RicardoVaranda:patch-1

Conversation

@RicardoVaranda

@RicardoVaranda RicardoVaranda commented Sep 19, 2017

Copy link
Copy Markdown
Contributor

@nguniversal/module-map-ngfactory-loader has been release and with it, support for Lazy loading!

This PR updates the existing story to guide users how they can use this functionality.

Closes: #7706

@filipesilva

Copy link
Copy Markdown
Contributor

@alxhub can you have a look?

// Build to dist-server instead of dist. This prevents
// client and server builds from overwriting each other.
"outDir": "dist-server",
"outDir": "dist/dist-server",

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Bad idea to put the Universal files in the same directory as normal build, I've tried this option and it causes issues. Better stay in another directory.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the suggestion @cyrilletuzi I'll apply those changes now 😄 Just out of curiosity what are the issues caused by having it within the dist?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't remember exactly, but good chances it was a problem of wrong import paths (as it's in a subdirectory) or a conflict on some building steps as files are in the same main directory.

@nguniversal/module-map-ngfactory-loader has been release and with it, support for Lazy loading!

This PR updates the existing story to guide users how they can use this functionality.
@MarkPieszak

Copy link
Copy Markdown
Contributor

Didn't see this PR in there, but added TypeScript (changes similar to these) and other fixes @vikerman requested here: #7796

@genyklemberg

Copy link
Copy Markdown

I used this answer, it is about firebase deploy, but main logic is simple https://hackernoon.com/deploy-angular-universal-w-firebase-ad70ea2413a1

@RicardoVaranda

Copy link
Copy Markdown
Contributor Author

Overwritten by #7803

@angular-automatic-lock-bot

Copy link
Copy Markdown

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 Sep 12, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Lazy Loading with Universal

7 participants