Skip to content

Conversation

@skrustev
Copy link
Member

No description provided.

@dkamburov dkamburov requested a review from Copilot February 5, 2026 08:28
@dkamburov dkamburov marked this pull request as ready for review February 5, 2026 08:28
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR migrates the localization samples to use a new localization API (registerI18n and setCurrentI18n) instead of the legacy approach. The old samples are preserved under a separate "localization-samples-old" directory with a "Legacy" designation, while the main localization sample is updated to demonstrate the new API.

Changes:

  • Updated the main localization sample to use the new registerI18n and setCurrentI18n APIs
  • Moved legacy localization samples to a separate directory with a new route
  • Added provideIgniteIntl() to application configuration
  • Updated dependencies to use alpha versions that support the new localization API

Reviewed changes

Copilot reviewed 11 out of 21 changed files in this pull request and generated 4 comments.

Show a summary per file
File Description
src/app/services/services.routes.ts Updated imports and added route for legacy localization sample
src/app/services/services-routes-data.ts Added display name for legacy localization route
src/app/services/localization-samples/localization-all-resources/localization-all-resources.component.ts Refactored to use new registerI18n and setCurrentI18n APIs, removed Angular locale registration
src/app/services/localization-samples-old/localization-all-resources/localization-all-resources.component.ts New file preserving the legacy localization implementation
src/app/services/localization-samples-old/localization-all-resources/localization-all-resources.component.scss New file with styles for legacy component
src/app/services/localization-samples-old/localization-all-resources/localization-all-resources.component.html New file with template for legacy component
src/app/app.config.ts Added provideIgniteIntl() provider for new localization system
package.json Updated igniteui-angular packages to alpha versions and added igniteui-i18n-resources dependency
live-editing/configs/LocalizationConfigGenerator.ts Updated to use BaseAppAngularLocaleConfig for legacy samples
live-editing/configs/BaseConfig.ts Added provideIgniteIntl() provider and duplicate HammerModule entry
live-editing/configs/BaseConfig-AngularLocale.ts New config file for legacy localization samples without new provider

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

skrustev and others added 3 commits February 5, 2026 10:33
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@dkamburov dkamburov merged commit 2a49e69 into vnext Feb 5, 2026
9 checks passed
@dkamburov dkamburov deleted the skrastev/new-localization branch February 5, 2026 09:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants