Skip to content
This repository has been archived by the owner on Sep 4, 2023. It is now read-only.

Commit

Permalink
Update bergamot translator to v0.4.1 (#111)
Browse files Browse the repository at this point in the history
* Tested latest bergamot artifacts

* Removed "once-release" from package.json file

* Updated engine registry for testing

* constructTranslationService is not an async function

 - Removed "await" keyword at the callsite

* Imported latest changes from translator into translation worker

 - Imported all api changes pertaining to bergamot translator engine

* Updated the comment for constructing ResponseOptions

* Using v0.4.1 artifacts for the engine

* Upated bergamot-translator-worker.js file with latest artifacts

* Fixed linting errors

* Fixed end-to-end test failures

 - Changed input Spanish sentence and modified the expected result
   as well
  • Loading branch information
abhi-agg committed Feb 19, 2022
1 parent 3821e90 commit af8943c
Show file tree
Hide file tree
Showing 6 changed files with 523 additions and 487 deletions.
6 changes: 5 additions & 1 deletion extension/controller/translation/Translation.js
Original file line number Diff line number Diff line change
Expand Up @@ -83,7 +83,11 @@ class Translation {
translate(translationMessage) {

if (translationMessage.type === "outbound") {
// if the message is from outbound translations, we skip the line.

/*
* if the message is from outbound translations, we skip queuing it and
* send for translation immediately
*/
if (this.translationWorker) {
this.translationWorker.postMessage([
"translate",
Expand Down
Loading

0 comments on commit af8943c

Please sign in to comment.