Skip to content

feat(@angular-devkit/build-angular): watch i18n translation files with dev server #19888

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

Merged
merged 1 commit into from
Nov 29, 2021

Conversation

clydin
Copy link
Member

@clydin clydin commented Jan 27, 2021

When using i18n with the dev server, the translation files will now be linked as a dependency to any file containing translated text. This allows translation files to be watched and the application to be rebuilt using the changed translation files.

Closes #16341

@clydin clydin added the target: minor This PR is targeted for the next minor release label Jan 27, 2021
@google-cla google-cla bot added the cla: yes label Jan 27, 2021
@clydin clydin force-pushed the i18n-serve-translation-file-watch branch 2 times, most recently from b847204 to 5bc5e61 Compare January 30, 2021 01:41
@clydin clydin force-pushed the i18n-serve-translation-file-watch branch 2 times, most recently from f5f1f7b to dcd4439 Compare March 2, 2021 20:13
@clydin clydin force-pushed the i18n-serve-translation-file-watch branch 5 times, most recently from 55a0a8e to 2ddef8d Compare November 24, 2021 20:56
@clydin clydin removed the state: WIP label Nov 24, 2021
@alan-agius4 alan-agius4 added the action: merge The PR is ready for merge by the caretaker label Nov 25, 2021
…h dev server

When using i18n with the dev server, the translation files will now be linked as a dependency to any file containing translated text. This allows translation files to be watched and the application to be rebuilt using the changed translation files.

Closes angular#16341
@clydin clydin force-pushed the i18n-serve-translation-file-watch branch from 2ddef8d to f19208a Compare November 25, 2021 12:25
@dgp1130 dgp1130 merged commit 9300545 into angular:master Nov 29, 2021
@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 30, 2021
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.

ng serve should watch i18n messages files
3 participants