Skip to content

feat(migrations): Add migration for functional Route guards and resolvers#49338

Closed
atscott wants to merge 1 commit into
angular:mainfrom
atscott:migrationfunctional
Closed

feat(migrations): Add migration for functional Route guards and resolvers#49338
atscott wants to merge 1 commit into
angular:mainfrom
atscott:migrationfunctional

Conversation

@atscott

@atscott atscott commented Mar 6, 2023

Copy link
Copy Markdown
Contributor

This migration updates Route definitions to use functions instead of class and InjectionToken references in the guard and resolver properties. This is accomplished by using the recently added mapper functions.

@atscott atscott added target: major This PR is targeted for the next major release area: migrations Issues related to `ng update`/`ng generate` migrations labels Mar 6, 2023
@ngbot ngbot Bot added this to the Backlog milestone Mar 6, 2023
@angular-robot angular-robot Bot added the detected: feature PR contains a feature commit label Mar 6, 2023
@atscott atscott force-pushed the migrationfunctional branch 3 times, most recently from c3b6e14 to 5bed1ae Compare March 6, 2023 21:39
…vers

This migration updates `Route` definitions to use functions instead of
class and `InjectionToken` references in the guard and resolver
properties. This is accomplished by using the recently added mapper
functions.
@JoostK

JoostK commented Oct 2, 2024

Copy link
Copy Markdown
Member

Closing for now, we can always revive if needed, likely as optional refactor schematic (ng generate) than an update (ng update)

@JoostK JoostK closed this Oct 2, 2024
@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 Nov 2, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

area: migrations Issues related to `ng update`/`ng generate` migrations detected: feature PR contains a feature commit target: major This PR is targeted for the next major release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants