Skip to content

Conversation

@GavinCLY
Copy link
Contributor

No description provided.

@chengzeyi
Copy link
Member

@codex Improve this PR. Make sure everything is in English.

@chatgpt-codex-connector
Copy link

Summary

  • Restricted i18n resources and the language list to English-only.
  • Locked the language preference to English and persist it in storage for consistency.
  • Screenshot not captured because npm install failed (network error downloading onnxruntime), so the dev server could not be started.

Testing

  • ⚠️ npm install (failed: ENETUNREACH while downloading onnxruntime)

View task →

@chengzeyi
Copy link
Member

@codex Improve this PR. Change all Chinese comments to English.

@chatgpt-codex-connector
Copy link

Summary

  • Translated remaining Chinese code comments to English across the browser electronAPI mock and shared utilities.
  • Clarified comment phrasing for browser-only behaviors in the web mock.

Testing

  • ⚠️ Not run (not requested).

View task →

@chengzeyi chengzeyi merged commit 14b69b6 into main Dec 27, 2025
3 checks passed
@chengzeyi chengzeyi deleted the web branch December 27, 2025 16:20
@chengzeyi chengzeyi restored the web branch December 28, 2025 04:05
@chengzeyi
Copy link
Member

@codex Make the deploy-to-server only happens in tagged build, not nightly

@chatgpt-codex-connector
Copy link

Summary

  • Removed the deploy-to-server job from the nightly workflow.
  • Added deploy-to-server to the tagged build workflow and gated it on successful release.

Testing

  • ⚠️ Not run (workflow change only).

View task →

@chengzeyi chengzeyi deleted the web branch December 28, 2025 04:17
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.

3 participants