Skip to content

Chinese translation for In the News page#1381

Open
EmmaLi1 wants to merge 1 commit into
saayam-for-all:testfrom
EmmaLi1:zh-news-translation
Open

Chinese translation for In the News page#1381
EmmaLi1 wants to merge 1 commit into
saayam-for-all:testfrom
EmmaLi1:zh-news-translation

Conversation

@EmmaLi1
Copy link
Copy Markdown

@EmmaLi1 EmmaLi1 commented Apr 6, 2026

I have added Chinese translation for the "In the News" page in news.json and common.json

@EmmaLi1 EmmaLi1 requested a review from a team as a code owner April 6, 2026 20:54
Copilot AI review requested due to automatic review settings April 6, 2026 20:54
@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 6, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
see 13 files with indirect coverage changes

🚀 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.

Copy link
Copy Markdown

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

Adds Simplified Chinese (zh) localization strings intended for the “In the News” page, expanding the Chinese locale coverage in the i18n layer.

Changes:

  • Added zh translations for the News/In-the-News page content (title/subtitle + 10 story titles/descriptions).
  • Added a Chinese label for IN_THE_NEWS in the shared common.json namespace.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
src/common/i18n/locales/zh/news.json New Chinese translations for the news page’s headings and story content.
src/common/i18n/locales/zh/common.json Adds a Chinese string for the IN_THE_NEWS key.

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

Comment on lines +1 to +8
{
"TITLE": "新闻:我们的故事",
"SUBTITLE": "了解Saayam for All如何因其在扶持社区、赋能志愿者和构建包容性互助网络方面的工作而备受关注和认可。",
"DESCRIPTION": "从本地报道到全国性专题,了解我们的使命如何从平台延伸到更广阔的世界。",
"READ_MORE": "阅读更多",
"JOIN_TITLE": "想要加入我们?",
"JOIN_BODY": "与我们的社区交流,联系不同的公益组织!",
"JOIN_BUTTON": "加入社区",
Copy link

Copilot AI Apr 6, 2026

Choose a reason for hiding this comment

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

zh/news.json is added but Chinese resources don’t currently load the news namespace (in src/common/i18n/i18n.js, zh only includes common/auth/enums/... and is missing news). As a result, the In the News page will still fall back to English for Chinese. Please import ./locales/zh/news.json and wire it into the zh resources under the news namespace.

Copilot uses AI. Check for mistakes.
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.

2 participants