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
Fix bug 1604843: Add support for SYSTRAN Translate service #1658
Conversation
6037355
to
a347080
Compare
I glanced at the results on stage, and there's one where I don't know why, https://mozilla-pontoon-staging.herokuapp.com/de/firefox/all-resources/?string=192281. There's a leading |
@@ -0,0 +1,53 @@ | |||
# Generated by Django 2.2.13 on 2020-07-13 16:47 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Minor comment, maybe give this one a semantic name?
blank=True, | ||
help_text=""" | ||
SYSTRAN Profile UUID to specify the engine trained on the en-locale language pair. | ||
""", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We talked about this on the call, we shouldn't make admins enter this, if it can be avoided. You mentioned there would be an API to query this?
Not terribly bound to where we store this.
Including support for custom engines (profiles).
dfd63fb
to
649277e
Compare
The patch is deployed to stage. Systran is only enabled for German, French and Spanish (es and es-ES).
Followups: