Skip to content

E0d/i18n fixes #290

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

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

E0d/i18n fixes #290

wants to merge 4 commits into from

Conversation

e0d
Copy link

@e0d e0d commented Apr 11, 2025

Proposing a few i18n improvements.

Edward Zarecor added 4 commits April 11, 2025 15:47
This commit fixes an internationalization issue in the FileMetaDisplay component by replacing the
hardcoded 'Unknown' string with a proper internationalized message. The change:
1. Adds a new message definition in messages.js for unknown file size
2. Updates the FileMetaDisplay component to use FormattedMessage
3. Ensures consistent use of i18n patterns across the application

This helps ensure proper translation of all user-facing text in the application.
This commit adds proper internationalization to the 'Past due\!' message in the ProgressStep component by:
1. Adding a new message definition in ProgressBar/messages.js for the past due warning
2. Importing FormattedMessage and messages in the ProgressStep component
3. Replacing the hardcoded string with the internationalized message

This ensures the past due warning can be properly translated to different languages.
This commit adds proper internationalization to the title in the StudioViewTitle component by:
1. Adding a new message definition for blockTitle in the studio view messages file
2. Using formatMessage with the title as a parameter instead of directly rendering it

This change ensures the block title can be properly translated and follows the
project's internationalization standards.
Copy link

codecov bot commented Apr 11, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 89.29%. Comparing base (0d2ddde) to head (ede8ffe).

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #290   +/-   ##
=======================================
  Coverage   89.29%   89.29%           
=======================================
  Files         143      143           
  Lines        2205     2205           
  Branches      476      476           
=======================================
  Hits         1969     1969           
  Misses        222      222           
  Partials       14       14           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@e0d e0d added the create-sandbox open-craft-grove should create a sandbox environment from this PR label Apr 11, 2025
@e0d e0d changed the title E0d/i18n fixes E0d/i18n fixes Apr 11, 2025
@e0d e0d changed the title E0d/i18n fixes E0d/i18n fixes Apr 11, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
create-sandbox open-craft-grove should create a sandbox environment from this PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant