NotificationsYou must be signed in to change notification settings
Fork195
Star58

[GSOC 2025]: Github action for translation tracker#895

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Jump to bottom

Open

Divyansh013 wants to merge22 commits intoprocessing:main

base:main

Choose a base branch

fromDivyansh013:week2

Open

[GSOC 2025]: Github action for translation tracker#895

Divyansh013 wants to merge22 commits intoprocessing:mainfromDivyansh013:week2

Conversation

Copy link

Member

Divyansh013 commentedJul 6, 2025•
edited
Loading

test-local file detects local changes
index.js does manual triggering for issue creation
action works on all "push" events
outdated banner addition on missing translation pages

Divyansh013 added17 commits

June 10, 2025 01:03

Test: Add documentation line to circle.mdx for translation tracker te…

46edaf0

…sting

Add Week 1 translation tracker implementation

ba9b4ee

Test: Modify triangle.mdx for translation testing

cd94f24

Test: Modify accelerationX.mdx to test missing translation detection

ddebbf3

removed testing logs

1e13287

small fix

875a77c

post review changes

5170eb6

Week 2: Translation tracker with GitHub API and Hindi focus

56abeec

Test: Update English example for translation tracker testing

d9c98c6

Fix: Add automatic branch detection to translation tracker

1976d9d

week 2 complete

71e7049

Week 3: Test automated translation tracking

94159c8

- Updated translation-sync workflow for examples instead of reference- Added Week 3 test change to Shape Primitives description- Should trigger automated issue creation for outdated translations

Add manual trigger support to translation tracker workflow

f03b60d

- Added workflow_dispatch with scan_all and create_issues options- Can now manually trigger action from GitHub UI- Supports both normal and full-scan modes

Test: Manual trigger detection test

dea5e7c

- Modified description.mdx to test translation tracker- Should now detect 1 English example file change

Simplify translation tracker - clean output

486aded

- test-local.js: Only shows changed files from predefined test files- index.js: Production mode with minimal console output- Removed verbose logging and repository exploration- Clear distinction: test mode vs production mode with GitHub token

Add scan all files option to index.js

63753de

- SCAN_ALL=true environment variable to check all 61 example files- Simplified GitHub Actions workflow- Default: checks only changed files- Manual trigger: option to scan all files

final fix

9f63c9d

Copy link

Member

ksen0 commentedJul 15, 2025

Hi@Divyansh013 , this is looking great!

Based on checking the current issue created by this (Divyansh013#11) I have a couple of minor suggestions:

Move🔍 Compare changes to the Quick Links section, because it's not file-specific.
Potentially: next to thisone link to "Compare changes", could add a note like:ℹ️ You can see the changes in a specific file by going the the "Compare changes" link above, clicking on the "Files changed" tab, and navigating to the specific file.
Move "Change Type: English file was updated. Spanish (Español), Hindi (हिन्दी), Korean (한국어), Chinese Simplified (简体中文) translations may be outdated." into the Timeline - right now this is very far separated, but seems related?

lirenjie95 reviewed

Jul 17, 2025

View reviewed changes

.github/actions/translation-tracker/index.js OutdatedShow resolvedHide resolved

.github/actions/translation-tracker/index.jsShow resolvedHide resolved

.github/actions/translation-tracker/index.js OutdatedShow resolvedHide resolved

Divyansh013 added2 commits

August 11, 2025 21:35

temp commit

8de0a67

basic version complete

18bd739

Divyansh013 marked this pull request as ready for review

August 18, 2025 05:10

Copy link

MemberAuthor

Divyansh013 commentedAug 18, 2025

@ksen0 requesting your review here

Divyansh013 changed the title~~Basic functionality of translation tracker~~[GSOC 2025]: Github actions for translation tracker

Aug 18, 2025

Divyansh013 changed the title~~[GSOC 2025]: Github actions for translation tracker~~[GSOC 2025]: Github action for translation tracker

Aug 18, 2025

lirenjie95 reviewed

Aug 18, 2025

View reviewed changes

src/components/OutdatedTranslationBanner/index.astroShow resolvedHide resolved

ksen0 reviewed

Aug 19, 2025

View reviewed changes

.github/actions/translation-tracker/README.md OutdatedShow resolvedHide resolved

ksen0 reviewed

Aug 19, 2025

View reviewed changes

src/components/OutdatedTranslationBanner/index.astroShow resolvedHide resolved

src/components/OutdatedTranslationBanner/index.astro

		contribute:'अनुवाद में योगदान दें',
		},
		'es': {
		title:'Esta traducción podría estar desactualizada',

Copy link

Member

Choose a reason for hiding this comment

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

Hi@marioguzzzman! If/when you have time, could you check these Spanish translations please? Thanks so much!

Copy link

marioguzzzmanOct 14, 2025•
edited
Loading

Choose a reason for hiding this comment

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

The grammatically correct way would be to say "Ver la versión en inglés" but I think it works better and is more clean for an internet use context to leave it like: "Ver versión en inglés". So I would say the translation works correctly@ksen0!

Copy link

Member

ksen0Oct 14, 2025

Choose a reason for hiding this comment

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

Thanks!

lirenjie95 reviewed

Aug 21, 2025

View reviewed changes

src/layouts/ExampleLayout.astro

		let englishUrl=Astro.url.pathname;
		if (currentLocale!=='en') {
		// Replace /{locale}/ with / to get English URL
		englishUrl=englishUrl.replace(`/${currentLocale}/`,'/');

Copy link

Contributor

lirenjie95Aug 21, 2025

Choose a reason for hiding this comment

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

Question here: have you tried usingzh-Hans? It is a different use case fromes,hi andko. I’m wondering if the replace operation will succeed.
PS: Please check all places withcurrentLocale andreplace()

Copy link

MemberAuthor

Divyansh013Aug 21, 2025

Choose a reason for hiding this comment

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

Yes have checked zh-Hans and it is working in expected way. I will check in all places

readme updated and simplified scanning

7fb683c

Divyansh013 requested a review fromksen0

August 21, 2025 16:37

ksen0 reviewed

Aug 23, 2025

View reviewed changes

.github/actions/translation-tracker/README.md

		-Links to files and comparisons
		-Action checklist for translators
		-Proper labels (`needs translation`, `lang-es`, etc.)
		4. Generates manifest file for website banner system

Copy link

Member

ksen0Aug 23, 2025

Choose a reason for hiding this comment

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

Thanks so much for the README updates! As one last thing, please include here or ~line 112 a short explanation/example of thetranslation-status format.

.github/actions/translation-tracker/index.js

		constenglishCommit=commitInfo.english;
		consttranslationCommit=commitInfo.translation;

		return`## 🌍 Translation Update Needed

Copy link

Member

ksen0Aug 23, 2025

Choose a reason for hiding this comment

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

As a future improvement, this could be pulled out as a template file separate form the .js script.

.github/workflows/translation-sync.yml


		on:
		push:
		branches:[main, week2]

Copy link

Member

ksen0Aug 23, 2025

Choose a reason for hiding this comment

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

This is looking great and nearly ready for further testing. If we're now in step 1 of the below iterative deployment process, then to get to step 2, I think this should listen for push on branchonly for now

Finalize this PR tested on your fork, document any further improvements, etc.
create a new PR that contains only the action. This PR would target the main branch, but the action would listen for translation changes on the staging branch.
create a PR that targets the staging branch to add the UI. This will allow fine-tuning the tool.
Lastly, to make it go live, the process would be to marge the UI PR into main, and then change the action to listen on main rather than branch

.github/actions/translation-tracker/index.js OutdatedShow resolvedHide resolved

.github/actions/translation-tracker/index.js Outdated

		examples,
		};
		fs.writeFileSync(manifestPath,JSON.stringify(manifest,null,2),'utf8');
		console.log(`\n🗂️ Wrote translation manifest:${manifestPath}`);

Copy link

Member

ksen0Aug 23, 2025

Choose a reason for hiding this comment

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

Include current directory in output

.github/actions/translation-tracker/package.json OutdatedShow resolvedHide resolved

.github/actions/translation-tracker/README.md

		on:
		push:
		branches:[main, week2]
		paths:['src/content/examples/en/**']

Copy link

Member

ksen0Aug 23, 2025

Choose a reason for hiding this comment

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

This already looks generalized, which is great

.github/actions/translation-tracker/README.md OutdatedShow resolvedHide resolved

src/layouts/ExampleLayout.astro

		console.log('DEBUG: Current locale:',currentLocale);

		try {
		const manifestPath=path.join(process.cwd(),"public","translation-status","examples.json");

Copy link

Member

ksen0Aug 23, 2025

Choose a reason for hiding this comment

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

I think at this stage, it makes sense to keep this here, but in the future (or now, optionally, if you prefer) also generalize it as a component usable in other layouts.

review fix

034694a

Divyansh013 requested a review fromksen0

September 1, 2025 18:57

Divyansh013 mentioned this pull request

Sep 2, 2025

[GSOC 2025] : Action only Pull Request#953

Closed

updated manifest

7f55a56

Divyansh013 mentioned this pull request

Sep 8, 2025

[GSOC 2025] Translation Tracker github action#958

Merged

Labels

None yet

4 participants

Movatterモバイル変換

[GSOC 2025]: Github action for translation tracker#895

Are you sure you want to change the base?

[GSOC 2025]: Github action for translation tracker#895

Uh oh!

Conversation

Divyansh013 commentedJul 6, 2025• editedLoading Uh oh!There was an error while loading.Please reload this page.

Uh oh!

Uh oh!

ksen0 commentedJul 15, 2025

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Divyansh013 commentedAug 18, 2025

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

marioguzzzmanOct 14, 2025• editedLoading Uh oh!There was an error while loading.Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

Divyansh013 commentedJul 6, 2025•
edited
Loading

marioguzzzmanOct 14, 2025•
edited
Loading