Skip to content
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

refactor(router): Remove DeprecatedLoadChildren #48292

Closed

Conversation

atscott
Copy link
Contributor

@atscott atscott commented Nov 30, 2022

This option has already been removed from availability externally. Internally, the last use has been removed for the type on Route.loadChildren. More testing will be necessary to verify nothing relies on the other code paths.

This option has already been removed from availability externally. Internally, the last use has been removed
for the type on Route.loadChildren. More testing will be necessary to verify nothing relies on the other
code paths.
@atscott atscott added the target: patch This PR is targeted for the next patch release label Nov 30, 2022
@atscott
Copy link
Contributor Author

atscott commented Nov 30, 2022

Caretaker Note: Please ignore the google-internal-tests status. requires and update to a g3 patch to run presubmits

@atscott atscott 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 Nov 30, 2022
@atscott
Copy link
Contributor Author

atscott commented Nov 30, 2022

TGP. Waiting on 1 change to be submitted

@pullapprove pullapprove bot requested a review from alxhub November 30, 2022 01:06
Copy link
Contributor

@AndrewKushnir AndrewKushnir left a 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

Copy link
Contributor

@dylhunn dylhunn left a 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

@atscott atscott added the action: merge The PR is ready for merge by the caretaker label Nov 30, 2022
@atscott
Copy link
Contributor Author

atscott commented Nov 30, 2022

This PR was merged into the repository by commit d5aaf55.

@atscott atscott closed this in d5aaf55 Nov 30, 2022
atscott added a commit that referenced this pull request Nov 30, 2022
This option has already been removed from availability externally. Internally, the last use has been removed
for the type on Route.loadChildren. More testing will be necessary to verify nothing relies on the other
code paths.

PR Close #48292
sr5434 pushed a commit to sr5434/angular that referenced this pull request Dec 3, 2022
This option has already been removed from availability externally. Internally, the last use has been removed
for the type on Route.loadChildren. More testing will be necessary to verify nothing relies on the other
code paths.

PR Close angular#48292
@angular-automatic-lock-bot
Copy link

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 Dec 31, 2022
trekladyone pushed a commit to trekladyone/angular that referenced this pull request Feb 1, 2023
This option has already been removed from availability externally. Internally, the last use has been removed
for the type on Route.loadChildren. More testing will be necessary to verify nothing relies on the other
code paths.

PR Close angular#48292
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 merge: caretaker note Alert the caretaker performing the merge to check the PR for an out of normal action needed or note target: patch This PR is targeted for the next patch release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants