Problem/Motivation

Check All Translation Progress Action not updating status correctly. Targets can be added in the TMS site, so checking the status should actually make a round-trip.

This has been added to content already, but with no tests, and is missing for config and config entities.

Proposed resolution

Add test coverage for this operation.
Adapt config entities and config objects for performing as content. Add testing for that.

Remaining tasks

None

User interface changes

None.

API changes

Added \Drupal\lingotek\LingotekInterface::getDocumentTranslationStatuses
Added \Drupal\lingotek\Remote\LingotekApiInterface::getDocumentTranslationStatuses

Data model changes

None.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

penyaskito created an issue. See original summary.

penyaskito’s picture

Added tests and implemented for LingotekConfigTranslationService.

The last submitted patch, 2: 2881489-check-translation-statuses-2.only-tests.patch, failed testing.

penyaskito’s picture

Status: Needs review » Reviewed & tested by the community

Patches failed and succeeded as expected.

  • penyaskito committed 1de6503 on 8.x-1.x
    Issue #2881489 by penyaskito: Check All Translation Progress Action not...
  • penyaskito committed a8d17bc on 8.x-2.x
    Issue #2881489 by penyaskito: Check All Translation Progress Action not...
penyaskito’s picture

Status: Reviewed & tested by the community » Fixed

Committed 1de6503 and pushed to 8.x-1.x. Thanks!
Cherry picked a8d17bc and pushed to 8.x-2.x.

Status: Fixed » Closed (fixed)

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