Skip to content

refactor(upgrade): downgradeModule uses zone-based change detection b… - #63814

Closed
atscott wants to merge 3 commits into
angular:mainfrom
atscott:downgradeModuleZone
Closed

atscott wants to merge 3 commits into
angular:mainfrom
atscott:downgradeModuleZone

Conversation

@atscott

@atscott atscott commented Sep 15, 2025

Copy link
Copy Markdown
Contributor

…y default

To avoid the need for specifying provideZoneChangeDetection in any/all modules used with downgradedModule, this change adds the zone-based change detection providers by default.

…y default

To avoid the need for specifying `provideZoneChangeDetection` in any/all
modules used with `downgradedModule`, this change adds the zone-based change detection
providers by default.
@angular-robot angular-robot Bot added the area: upgrade Issues related to AngularJS → Angular upgrade APIs label Sep 15, 2025
@ngbot ngbot Bot added this to the Backlog milestone Sep 15, 2025
…y default

To avoid the need for specifying `provideZoneChangeDetection` in any/all
modules used with `downgradedModule`, this change adds the zone-based change detection
providers by default.
@atscott
atscott force-pushed the downgradeModuleZone branch 4 times, most recently from 0ee7966 to 558a294 Compare September 15, 2025 20:20
@atscott
atscott marked this pull request as ready for review September 15, 2025 20:23
@atscott
atscott force-pushed the downgradeModuleZone branch from 558a294 to 24911fa Compare September 15, 2025 20:27
@atscott atscott added the target: major This PR is targeted for the next major release label Sep 15, 2025
Comment thread packages/core/src/platform/platform_ref.ts
@atscott
atscott force-pushed the downgradeModuleZone branch from 24911fa to 426f17f Compare September 15, 2025 20:40
To avoid the need for specifying `provideZoneChangeDetection` in any/all
modules used with `downgradedModule`, this change adds the zone-based change detection
providers by default. This also adds the zone providers to
`UpgradeModule` by default as well.
@atscott
atscott force-pushed the downgradeModuleZone branch from 426f17f to 96682f8 Compare September 15, 2025 20:42
@atscott atscott added the action: merge The PR is ready for merge by the caretaker label Sep 15, 2025
thePunderWoman pushed a commit that referenced this pull request Sep 16, 2025
…y default (#63814)

To avoid the need for specifying `provideZoneChangeDetection` in any/all
modules used with `downgradedModule`, this change adds the zone-based change detection
providers by default.

PR Close #63814
thePunderWoman pushed a commit that referenced this pull request Sep 16, 2025
To avoid the need for specifying `provideZoneChangeDetection` in any/all
modules used with `downgradedModule`, this change adds the zone-based change detection
providers by default. This also adds the zone providers to
`UpgradeModule` by default as well.

PR Close #63814
@thePunderWoman

Copy link
Copy Markdown
Contributor

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

wildcardalice pushed a commit to wildcardalice/angular that referenced this pull request Sep 18, 2025
…y default (angular#63814)

To avoid the need for specifying `provideZoneChangeDetection` in any/all
modules used with `downgradedModule`, this change adds the zone-based change detection
providers by default.

PR Close angular#63814
wildcardalice pushed a commit to wildcardalice/angular that referenced this pull request Sep 18, 2025
…y default (angular#63814)

To avoid the need for specifying `provideZoneChangeDetection` in any/all
modules used with `downgradedModule`, this change adds the zone-based change detection
providers by default.

PR Close angular#63814
wildcardalice pushed a commit to wildcardalice/angular that referenced this pull request Sep 18, 2025
…r#63814)

To avoid the need for specifying `provideZoneChangeDetection` in any/all
modules used with `downgradedModule`, this change adds the zone-based change detection
providers by default. This also adds the zone providers to
`UpgradeModule` by default as well.

PR Close angular#63814
@angular-automatic-lock-bot

Copy link
Copy Markdown

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

This action has been performed automatically by a bot.

@angular-automatic-lock-bot angular-automatic-lock-bot Bot locked and limited conversation to collaborators Oct 17, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

action: merge The PR is ready for merge by the caretaker area: upgrade Issues related to AngularJS → Angular upgrade APIs target: major This PR is targeted for the next major release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants