Skip to content

refactor(core): deprecate withIncrementalHydration#68114

Merged
kirjs merged 1 commit intoangular:mainfrom
SkyZeroZx:refactor/hydration
Apr 13, 2026
Merged

refactor(core): deprecate withIncrementalHydration#68114
kirjs merged 1 commit intoangular:mainfrom
SkyZeroZx:refactor/hydration

Conversation

@SkyZeroZx
Copy link
Copy Markdown
Contributor

@SkyZeroZx SkyZeroZx commented Apr 9, 2026

Deprecates withIncrementalHydration() as it is no longer required. ( Considering that this is now the default behavior #68092 )

Updates API docs and runtime errors to reflect the new default and guide opt-out.

@pullapprove pullapprove bot requested a review from thePunderWoman April 9, 2026 18:30
@angular-robot angular-robot bot added the area: core Issues related to the framework runtime label Apr 9, 2026
Comment thread packages/platform-browser/src/hydration.ts Outdated
Deprecates `withIncrementalHydration()` as it is no longer required.

Updates API docs and runtime errors to reflect the new default and guide opt-out.
@SkyZeroZx SkyZeroZx force-pushed the refactor/hydration branch from 64f6460 to a551207 Compare April 9, 2026 18:50
@alan-agius4 alan-agius4 added the target: major This PR is targeted for the next major release label Apr 10, 2026
Copy link
Copy Markdown
Contributor

@thePunderWoman thePunderWoman left a comment

Choose a reason for hiding this comment

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

LGTM

reviewed-for: fw-general, public-api, angular-dev

Comment thread goldens/public-api/platform-browser/index.api.md
@pullapprove pullapprove bot requested review from kirjs and thePunderWoman April 10, 2026 15:15
@thePunderWoman thePunderWoman added the action: merge The PR is ready for merge by the caretaker label Apr 10, 2026
@ngbot ngbot bot added this to the Backlog milestone Apr 10, 2026
@thePunderWoman thePunderWoman added the merge: caretaker note Alert the caretaker performing the merge to check the PR for an out of normal action needed or note label Apr 10, 2026
@thePunderWoman
Copy link
Copy Markdown
Contributor

Caretaker note: this is safe to merge. The github bug is preventing dismissing unnecessary reviews.

@kirjs kirjs merged commit 0454d4c into angular:main Apr 13, 2026
23 of 25 checks passed
@kirjs
Copy link
Copy Markdown
Contributor

kirjs commented Apr 13, 2026

This PR was merged into the repository. The changes were merged into the following branches:

alan-agius4 added a commit to alan-agius4/angular-cli that referenced this pull request Apr 14, 2026
…m `provideClientHydration`

`withEventReplay` and incremental hydration are now enabled by default when using `provideClientHydration()`.

Since incremental hydration automatically depends on and enables event replay, explicitly passing `withEventReplay()` is no longer necessary.

See: angular/angular#68114
alan-agius4 added a commit to angular/angular-cli that referenced this pull request Apr 14, 2026
…m `provideClientHydration`

`withEventReplay` and incremental hydration are now enabled by default when using `provideClientHydration()`.

Since incremental hydration automatically depends on and enables event replay, explicitly passing `withEventReplay()` is no longer necessary.

See: angular/angular#68114
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

action: merge The PR is ready for merge by the caretaker area: core Issues related to the framework runtime merge: caretaker note Alert the caretaker performing the merge to check the PR for an out of normal action needed or note target: major This PR is targeted for the next major release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants