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

feat(demos): enable SSR for "Feature App in Feature App" demo and fix externals config #326

Merged
merged 2 commits into from
Feb 4, 2019

Conversation

unstubbable
Copy link
Member

No description provided.

@unstubbable unstubbable changed the title feat(demos): enable SSR for FA-in-FA demo feat(demos): enable SSR for "Feature App in Feature App" demo Feb 1, 2019
@unstubbable unstubbable changed the title feat(demos): enable SSR for "Feature App in Feature App" demo feat(demos): enable SSR for "Feature App in Feature App" demo and fix externals config Feb 4, 2019

// In a real-world integrator, instead of preloading a Feature App manually
// before rendering, the Async SSR Manager would be used to handle the
// asynchronous server-side rendering (see "server-side-rendering" demo).
Copy link
Member

Choose a reason for hiding this comment

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

In the demos README it is spelled: "Server-Side Rendering" demo

@@ -15,6 +15,9 @@ export default async function renderApp({
moduleLoader: loadCommonJsModule
});

// In a real-world integrator, instead of preloading a Feature App manually
// before rendering, the Async SSR Manager would be used to handle the
// asynchronous server-side rendering (see "server-side-rendering" demo).
Copy link
Member

Choose a reason for hiding this comment

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

"Server-Side Rendering" demo

@unstubbable unstubbable merged commit 0f13102 into master Feb 4, 2019
@unstubbable unstubbable deleted the fa-in-fa-ssr branch February 4, 2019 11:13
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