Motivation

  • Get rid of an outdated Context class in a connector code base.
  • Get rid of an org id and api key provider settings.

Possible solution

This patch removes an old SmartlingContextApi class and provides integration with the new class from api sdk. This patch doesn't contain updated api-sdk-php and vendor folders.

All tests are passed.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Loparev created an issue. See original summary.

Loparev’s picture

Issue summary: View changes
Loparev’s picture

tmgmt_smartling-context.patch - new context api integration patch
tmgmt_smartling-api-sdk-php-update.patch - updated sdk
tmgmt_smartling-context-2905671-3.patch - full patch

Loparev’s picture

Updated sdk (fixed issue with user agent)

Loparev’s picture

Edited file name generation

Loparev’s picture

Improved logging

  • Loparev committed d9b5d73 on 8.x-2.x
    Issue #2905671 by Loparev: API sdk update: use new ContextAPI class
    
Loparev’s picture

Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

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