-
Notifications
You must be signed in to change notification settings - Fork 27k
feat(platform-server): add provideServerSupport function to provide server capabilities to an application
#49380
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
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
jessicajaniuk
left a comment
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
AndrewKushnir
left a comment
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
AndrewKushnir
left a comment
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.
👍
AndrewKushnir
left a comment
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 theEnvironmentProvidersneeded to setup server rendering without NgModules.