Skip to content

fix(@angular-devkit/build-angular): ensure $localize calls are replaced in watch mode - #22461

Merged
dgp1130 merged 1 commit into
angular:masterfrom
alan-agius4:i18n-watch
Jan 12, 2022
Merged

fix(@angular-devkit/build-angular): ensure $localize calls are replaced in watch mode#22461
dgp1130 merged 1 commit into
angular:masterfrom
alan-agius4:i18n-watch

Conversation

@alan-agius4

Copy link
Copy Markdown
Collaborator

When translations is undefined $localize calls are not replaced.

Closes #22435

@alan-agius4 alan-agius4 added the target: patch This PR is targeted for the next patch release label Jan 10, 2022
@alan-agius4
alan-agius4 requested review from clydin and dgp1130 and removed request for clydin January 10, 2022 14:37
@alan-agius4 alan-agius4 added the action: merge The PR is ready for merge by the caretaker label Jan 11, 2022
@dgp1130

dgp1130 commented Jan 12, 2022

Copy link
Copy Markdown
Collaborator

This has a merge conflict with 13.1.x. I'll merge to master for now, but it seems like we'll need another PR to land this in patch. Alternatively we could just wait until 13.2.0, but I'll leave that up to you @alan-agius4.

@dgp1130 dgp1130 added target: minor This PR is targeted for the next minor release and removed target: patch This PR is targeted for the next patch release labels Jan 12, 2022
@dgp1130
dgp1130 merged commit 426ddb6 into angular:master Jan 12, 2022
@alan-agius4
alan-agius4 deleted the i18n-watch branch January 12, 2022 18:44
@dgp1130

dgp1130 commented Jan 12, 2022

Copy link
Copy Markdown
Collaborator

Cherry picked to the patch branch in 6a617ff.

@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 Feb 12, 2022
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 target: minor This PR is targeted for the next minor release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Development with ng serve gets "stuck" with $localize error when using i18n attributes and localize option set to sourceLocale

2 participants