Skip to content

fix(@angular/ssr): ensure loadChildren runs in correct injection context during route extraction #30587

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

alan-agius4
Copy link
Collaborator

Ensure that loadChildren functions are executed within the route’s specific injection context when extracting routes, aligning behavior with Angular’s dependency injection model.

More context: angular/angular#62133

Closes #29483


Blocking until the next version of @angular/router is on NPM.

@alan-agius4 alan-agius4 added state: blocked target: minor This PR is targeted for the next minor release labels Jun 24, 2025
@alan-agius4 alan-agius4 requested a review from dgp1130 June 24, 2025 11:35
@alan-agius4 alan-agius4 added action: review The PR is still awaiting reviews from at least one requested reviewer and removed area: @angular/ssr labels Jun 24, 2025
…ntext during route extraction

Ensure that `loadChildren` functions are executed within the route’s specific injection context when extracting routes, aligning behavior with Angular’s dependency injection model.

More context: angular/angular#62133

Closes angular#29483
@alan-agius4 alan-agius4 removed the request for review from dgp1130 June 24, 2025 14:26
@alan-agius4 alan-agius4 removed the action: review The PR is still awaiting reviews from at least one requested reviewer label Jun 24, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: @angular/ssr state: blocked target: minor This PR is targeted for the next minor release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Bug] Route-level Environment Initializer Skipped During AppEngine Route Discovery
2 participants