feat(platform-browser): enable HTTP request caching when using provideClientHydration
#49699
Conversation
There was a problem hiding this comment.
Are HydrationFeature and HydrationFeatureKind really needed to be part of the public API? In @angular/common/http I noticed they are but was wondering what they are needed.
There was a problem hiding this comment.
I followed the typings that we have in the Router, but I'm ok with this option too 👍
There was a problem hiding this comment.
Should they still be exported?
There was a problem hiding this comment.
Yeah, we need to export them.
9c21736 to
1541f5d
Compare
a6ede8b to
df6bd17
Compare
|
Caretaker note: this requires a G3 patch and BUILD file to be updated. http://cl/521719612 The broken target seems to be caused by other changes in the presubmit CL. |
AndrewKushnir
left a comment
There was a problem hiding this comment.
LGTM, just a quick question on the API.
There was a problem hiding this comment.
I followed the typings that we have in the Router, but I'm ok with this option too 👍
…deClientHydration`
This commit adds support by default for HTTP caching when using `provideClientHydration`. Users can opt-out of this behaviour by using the `withoutHttpTransferCache` feature.
```ts
import {
bootstrapApplication,
provideClientHydration,
withNoHttpTransferCache,
} from '@angular/platform-browser';
// ...
bootstrapApplication(RootCmp, {
providers: [provideClientHydration(withNoHttpTransferCache())]
});
```
df6bd17 to
9e4365c
Compare
AndrewKushnir
left a comment
There was a problem hiding this comment.
@alan-agius4 looks great! 👍
AndrewKushnir
left a comment
There was a problem hiding this comment.
Reviewed-for: public-api
jessicajaniuk
left a comment
There was a problem hiding this comment.
reviewed-for: fw-core, fw-http, fw-platform-server, public-api
|
Caretaker notes:
|
|
This PR was merged into the repository by commit 81e7d15. |
|
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 support by default for HTTP caching when using
provideClientHydration. Users can opt-out of this behaviour by using thewithoutHttpTransferCachefeature.