Problem/Motivation

Coveo Labs has removed the PHP SDK that is a dependency of this module.

Proposed resolution

Re-engineer what we used of this functionality here in the module.

Remaining tasks

Usage:

  • SeachApiClientContext
    • Coveo\Search\SDK\SDKPushPHP\PushApiEndpoint
  • SearchApiCoveoBackend
    • Coveo\Search\SDK\SDKPushPHP\Document;
    • Coveo\Search\SDK\SDKPushPHP\PushApiEndpoint;
  • SearchApiCoveoClient
    • Coveo\Search\SDK\SDKPushPHP\Document;
    • Coveo\Search\SDK\SDKPushPHP\Push;

Which means that we use these classes:

  1. Coveo\Search\SDK\SDKPushPHP\Document;
  2. Coveo\Search\SDK\SDKPushPHP\PushApiEndpoint
  3. Coveo\Search\SDK\SDKPushPHP\Push;

User interface changes

API changes

Data model changes

Issue fork search_api_coveo-3407209

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

FatherShawn created an issue. See original summary.

fathershawn’s picture

fathershawn
Primary language English
Location New York
commented
Issue summary: View changes
fathershawn’s picture

fathershawn
Primary language English
Location New York
commented
Assigned: Unassigned » fathershawn
Status: Active » Needs review

Initial refactoring pushed

FatherShawn opened merge request !14

fathershawn’s picture

fathershawn
Primary language English
Location New York
commented

Integrated the functionality of the SDK into our SearchAPICoveoClient class and added a test. Merging to the dev branch so that we can easily test manually against a Coveo sandbox index.

FatherShawn credited aancaa.

fathershawn’s picture

fathershawn
Primary language English
Location New York
commented

  • FatherShawn committed e8218db1 on 1.0.x 
    Issue #3407209 by FatherShawn, aancaa: Replace Coveo Push API
fathershawn’s picture

fathershawn
Primary language English
Location New York
commented
Status: Needs review » Fixed

FatherShawn opened merge request !16

  • FatherShawn committed 847b5b87 on 1.0.x 
    Issue #3407209 by FatherShawn: bugs uncovered in manual testing vs coveo...

Status: Fixed » Closed (fixed)

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