-
Notifications
You must be signed in to change notification settings - Fork 24.8k
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(platform-server): add provideServerSupport
function to provide server capabilities to an application
#49380
Conversation
e48d0ae
to
b68d675
Compare
… server capabilities to an application This commit adds the `provideServerSupport()` function that returns the `EnvironmentProviders` needed to setup server rendering without NgModules.
b68d675
to
0cc694e
Compare
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.
reviewed-for: fw-platform-server, public-api
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.
@alan-agius4 I think it'd be great to avoid the usage of importProvidersFrom
and just use the functions that return providers directly.
We could also update the renderApplication
providers here to use the new method and replace BrowserModule.withServerTransition
with an APP_ID
provider. WDYT?
angular/packages/platform-server/src/utils.ts
Lines 242 to 243 in 86fc4d3
importProvidersFrom(BrowserModule.withServerTransition({appId})), | |
importProvidersFrom(ServerModule), |
…provide server capabilities to an application
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.
👍
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.
Reviewed-for: public-api
This PR was merged into the repository by commit 056d680. |
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. |
This commit adds the
provideServerSupport()
function that returns theEnvironmentProviders
needed to setup server rendering without NgModules.