Skip to content

Conversation

@its-mitesh-kumar
Copy link
Contributor

@its-mitesh-kumar its-mitesh-kumar commented Nov 26, 2025

Description

Adding internationalization (i18n) support with German, French, Italian, Spanish and Japanese translations.

##Fixed
Fixing https://issues.redhat.com/browse/RHIDP-9721

UI after changes

Screen.Recording.2025-11-26.at.11.mp4
  • A changeset describing the change and affected packages. (more info)
  • Added or updated documentation
  • Tests for new functionality and regression tests for bug fixes
  • Screenshots attached (for UI changes)
  • All your commits have a Signed-off-by line in the message. (more info)

Signed-off-by: its-mitesh-kumar <itsmiteshkumar98@gmail.com>
Signed-off-by: its-mitesh-kumar <itsmiteshkumar98@gmail.com>
@backstage-goalie
Copy link
Contributor

Changed Packages

Package Name Package Path Changeset Bump Current Version
@backstage-community/plugin-acr workspaces/acr/plugins/acr patch v1.19.0

@its-mitesh-kumar its-mitesh-kumar changed the title feat(jfrog-artifactory): adding support for internationalization (i18n) feat(acr): adding support for internationalization (i18n) Nov 26, 2025
Copy link
Contributor

@invincibleJai invincibleJai left a comment

Choose a reason for hiding this comment

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

/lgtm
/approve

import { ManifestDigestChip } from './ManifestDigestChip';

export const columns: TableColumn<TagRow>[] = [
export const getColumns = (
Copy link
Member

Choose a reason for hiding this comment

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

maybe this is better suited as a useMemo now?

const title = `Azure Container Registry Repository: ${image}`;
const { t } = useTranslation();
const title = t(
'page.title' as keyof typeof acrTranslationRef.T,
Copy link
Member

Choose a reason for hiding this comment

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

Bookmarks plugin has an example of useTranslation that removes the need for these type assertions if you want that

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.

4 participants