Make microsoft translator available for continuous jobs by implementing ContinuousTranslatorInterface.

CommentFileSizeAuthor
#12 interdiff-2680203-12.txt3.48 KBthenchev
#12 implement-2680203-12.patch8.11 KBthenchev
#9 implement-continuoustranslatorInterface-2680203-9.patch7.06 KBthenchev
#7 implement_ContinuousTranslatorInterface-2676154-13.patch7.27 KBthenchev
#7 interdiff.txt2.31 KBthenchev
#3 implement-continuoustranslatorInterface-2680203-3.patch7.06 KBthenchev
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Denchev created an issue. See original summary.

thenchev’s picture

thenchev CreditAttribution: thenchev at MD Systems GmbH commented
Issue tags: -Implement ContinuousTranslatorInterface
thenchev’s picture

thenchev CreditAttribution: thenchev at MD Systems GmbH commented
FileSize
implement-continuoustranslatorInterface-2680203-3.patch7.06 KB
thenchev’s picture

thenchev CreditAttribution: thenchev at MD Systems GmbH commented
Status: Active » Needs review

Status: Needs review » Needs work

The last submitted patch, 3: implement-continuoustranslatorInterface-2680203-3.patch, failed testing.

mbovan’s picture

mbovan CreditAttribution: mbovan at MD Systems GmbH commented

Tested manually, tests are passing locally, looks good to me.

thenchev’s picture

thenchev CreditAttribution: thenchev at MD Systems GmbH commented
Status: Needs work » Needs review
FileSize
interdiff.txt2.31 KB
implement_ContinuousTranslatorInterface-2676154-13.patch7.27 KB

Fixed the messed up label and some cleanup.

Status: Needs review » Needs work

The last submitted patch, 7: implement_ContinuousTranslatorInterface-2676154-13.patch, failed testing.

thenchev’s picture

thenchev CreditAttribution: thenchev at MD Systems GmbH commented
Status: Needs work » Needs review
FileSize
implement-continuoustranslatorInterface-2680203-9.patch7.06 KB

Sorry uploaded the wrong patch here re-uploading the one before.

Status: Needs review » Needs work

The last submitted patch, 9: implement-continuoustranslatorInterface-2680203-9.patch, failed testing.

Berdir’s picture

Berdir
Primary language German
Location Switzerland
CreditAttribution: Berdir at MD Systems GmbH commented

Same issues as #2676154: Implement ContinuousTranslatorInterface. But, both tests are passing now :)

thenchev’s picture

thenchev CreditAttribution: thenchev at MD Systems GmbH commented
Status: Needs work » Needs review
FileSize
implement-2680203-12.patch8.11 KB
interdiff-2680203-12.txt3.48 KB

Synced with #2676154: Implement ContinuousTranslatorInterface

  • Berdir committed c6c215f on 8.x-1.x authored by Denchev 
    Issue #2680203 by Denchev: Implement ContinuousTranslatorInterface
Berdir’s picture

Berdir
Primary language German
Location Switzerland
CreditAttribution: Berdir at MD Systems GmbH commented
Status: Needs review » Fixed

Thanks, committed.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.