Skip to content

Remove SYSTRAN machine translation integration (#4180)#4181

Merged
mathjazz merged 2 commits into
mozilla:mainfrom
mathjazz:remove-systran-4180
Jun 1, 2026
Merged

Remove SYSTRAN machine translation integration (#4180)#4181
mathjazz merged 2 commits into
mozilla:mainfrom
mathjazz:remove-systran-4180

Conversation

@mathjazz
Copy link
Copy Markdown
Collaborator

@mathjazz mathjazz commented May 29, 2026

Fix #4180.

SYSTRAN integration is no longer used.

Remove the view, URL, frontend fetch/component, Locale model fields (and add matching migration), API field, SYSTRAN_TRANSLATE_API_KEY family of settings, the data-is-systran-translate-supported flag passed to the translate view, and the l10n strings.

Translation.MachinerySource.SYSTRAN_TRANSLATE is intentionally retained so existing translations attributed to SYSTRAN remain valid; only the live integration is removed.

mathjazz added 2 commits May 29, 2026 21:37
SYSTRAN integration is no longer used. Remove the view, URL, frontend
fetch/component, Locale model fields (and add matching migration),
API field, SYSTRAN_TRANSLATE_API_KEY family of settings, the
data-is-systran-translate-supported flag passed to the translate view,
and the l10n strings.

Translation.MachinerySource.SYSTRAN_TRANSLATE is intentionally retained
so existing translations attributed to SYSTRAN remain valid; only the
live integration is removed.
docker/config/server.env.template > .env
# Run collectstatic with minimal dependencies, skipping the actual front-end build
- uses: actions/setup-node@48b55a011bda9f5d6aeb4c2d9c7362e8dae4041e # v6.4.0
with: { node-version: '24' }
Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

This is a fix for regression probably introduced in #4138.

@mathjazz mathjazz requested a review from eemeli May 29, 2026 20:04
@mathjazz mathjazz merged commit 5a0c11f into mozilla:main Jun 1, 2026
8 checks passed
@mathjazz mathjazz deleted the remove-systran-4180 branch June 1, 2026 12:51
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.

Drop SYSTRAN support

2 participants