Skip to content

Conversation

@jomcarvajal
Copy link
Contributor

@jomcarvajal jomcarvajal commented Dec 24, 2025

@TomWoodward TomWoodward temporarily deployed to rex-web-core-1455-vb36xzfvxvug December 24, 2025 13:59 Inactive
@TomWoodward TomWoodward temporarily deployed to rex-web-core-1455-vb36xzfvxvug December 24, 2025 14:09 Inactive
@TomWoodward TomWoodward temporarily deployed to rex-web-core-1455-vb36xzfvxvug December 24, 2025 14:26 Inactive
@TomWoodward TomWoodward temporarily deployed to rex-web-core-1455-vb36xzfvxvug December 24, 2025 14:58 Inactive
@TomWoodward TomWoodward temporarily deployed to rex-web-core-1455-vb36xzfvxvug December 24, 2025 15:10 Inactive
@TomWoodward TomWoodward temporarily deployed to rex-web-core-1455-vb36xzfvxvug December 24, 2025 15:41 Inactive
@jomcarvajal jomcarvajal marked this pull request as ready for review December 24, 2025 15:54
@jomcarvajal jomcarvajal requested a review from a team as a code owner December 24, 2025 15:54
@jomcarvajal jomcarvajal requested a review from Dantemss December 24, 2025 15:54
@Dantemss Dantemss requested a review from Copilot January 5, 2026 12:34
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 adds missing translations for "Content Warning" text across English, Spanish, and Polish locales. The changes implement internationalization support by introducing a new translation key i18n:content-warning:heading:aria-label and updating the LoginGate and ContentWarning components to use translated text instead of hardcoded English strings.

  • Added translation key i18n:content-warning:heading:aria-label to all three language files (en, es, pl)
  • Updated createIntl function to support optional message overrides for testing purposes
  • Modified LoginGate and ContentWarning components to use the internationalization API
  • Updated corresponding test files to accommodate the internationalization changes

Reviewed changes

Copilot reviewed 9 out of 9 changed files in this pull request and generated 6 comments.

Show a summary per file
File Description
src/app/messages/en/messages.json Added English translation "Content Warning" for the new translation key
src/app/messages/es/messages.json Added Spanish translation "Advertencia de contenido" for the new translation key
src/app/messages/pl/messages.json Added Polish translation "Ostrzeżenie o treści" for the new translation key
src/app/messages/createIntl.ts Enhanced to accept optional newMessages parameter for test message overrides
src/app/content/components/LoginGate.tsx Replaced hardcoded "Content Warning" string with internationalized version using useIntl
src/app/content/components/ContentWarning.tsx Replaced hardcoded "Content warning" string with internationalized version using useIntl
src/app/content/components/LoginGate.spec.tsx Updated test to mock useIntl and added IntlProvider setup
src/app/content/components/ContentWarning.spec.tsx Updated tests to mock useIntl and added IntlProvider setup
src/app/content/highlights/components/Highlights.spec.tsx Updated tests to use new newMessages parameter of createIntl function

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

"i18n:toolbar:textresizer:button:increase:aria-label": "Zwiększ rozmiar tekstu",
"i18n:toolbar:textresizer:popup:heading": "Rozmiar tekstu"
"i18n:toolbar:textresizer:popup:heading": "Rozmiar tekstu",
"i18n:content-warning:heading:aria-label": "Ostrzeżenie o treści"
Copy link
Member

Choose a reason for hiding this comment

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

This translation should probably be confirmed by our contact at Katalyst

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants