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

fix(@angular/build): replace deprecation of i18n.baseHref with a warning #29429

Merged
merged 1 commit into from
Jan 21, 2025

Conversation

alan-agius4
Copy link
Collaborator

In certain scenarios, users build applications with the same baseHref when using i18n, primarily for deploying localized applications across multiple domains. To address this, we are removing the deprecation of i18n.baseHref and will revisit potential options as part of #29111

Instead of deprecating i18n.baseHref, we now issue a warning when it is used with SSR, as this may lead to undefined behavior.

Closes #29396

@alan-agius4 alan-agius4 changed the title fix(@angular/build): replace deprecation of i18n.baseHref fix(@angular/build): remove deprecation of i18n.baseHref Jan 21, 2025
…rning

In certain scenarios, users build applications with the same `baseHref` when using i18n, primarily for deploying localized applications across multiple domains. To address this, we are removing the deprecation of `i18n.baseHref` and will revisit potential options as part of angular#29111

Instead of deprecating `i18n.baseHref`, we now issue a warning when it is used with SSR, as this may lead to undefined behavior.

Closes angular#29396
@alan-agius4 alan-agius4 changed the title fix(@angular/build): remove deprecation of i18n.baseHref fix(@angular/build): replace deprecation of i18n.baseHref with a warning Jan 21, 2025
@alan-agius4 alan-agius4 requested a review from clydin January 21, 2025 13:53
@alan-agius4 alan-agius4 added action: review The PR is still awaiting reviews from at least one requested reviewer target: patch This PR is targeted for the next patch release labels Jan 21, 2025
@alan-agius4 alan-agius4 added action: merge The PR is ready for merge by the caretaker and removed action: review The PR is still awaiting reviews from at least one requested reviewer labels Jan 21, 2025
@alan-agius4 alan-agius4 merged commit 8535c11 into angular:main Jan 21, 2025
33 checks passed
@alan-agius4 alan-agius4 deleted the base-href-i18n branch January 21, 2025 15:36
@alan-agius4
Copy link
Collaborator Author

The changes were merged into the following branches: main, 19.1.x

@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 Feb 21, 2025
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 area: @angular/build target: patch This PR is targeted for the next patch release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Do Not Deprecate baseHref in i18n Configuration
2 participants