1851609 - Add API endpoint to trigger analysis

Status: RESOLVED FIXED

(Firefox :: Shopping, task, P1)

Description

[Jared Hirsch [:jhirsch] (he/him) (Needinfo please)]

We've just heard that the fakespot API will support triggering analysis quite soon (within the next week or so).

Let's get the API design details and build the client-side integration in parallel, so this is ready for QA as soon as possible.

Comment 1

[Jared Hirsch [:jhirsch] (he/him) (Needinfo please)]

Marking 119 ship-blockers as P1s for easier tracking

Comment 2

[Fred Chasen [:fchasen]]

Attachment: Bug 1851609 - Add Analyze and Analysis Status Product APIs. r=#shopping-reviewers

• Adds the API endpoint and schema for analyze and analysis status.
• Adds requestAnalyze and requestAnalysisStatus methods for products.
• Updated pollForAnalysisCompleted to poll the analysis status api instead of analysis.
• After testing with the production api, polling now has an initial wait of 60s and will poll every 1s after that for 2 more minutes.
• Removed exponential backoff as the new api gives updated progress with the status, so better to get it consistently.
• Removed links to external analysis triggers.
• Update ShoppingSidebarChild updateContent to check if there is an analysis in progress and if so wait for it to finish before requesting the analysis.

NOTE: Now that pollForAnalysisCompleted has a status api to poll, it returns a status result. The analysis data will now need to be requested with requestAnalysis after the polling has finished.

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/c74fb301e2f4
Add Analyze and Analysis Status Product APIs. r=shopping-reviewers,jhirsch

Comment 4

[Narcis Beleuzu [:NarcisB]]

Backed out for bc failures on browser_shopping_integration.js

Backout link: https://hg.mozilla.org/integration/autoland/rev/c0ad2e66535c99622bade446611d036aa60bfcc8
Log link: https://treeherder.mozilla.org/logviewer?job_id=428935146&repo=autoland&lineNumber=15164

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/797c9afc3d0a
Add Analyze and Analysis Status Product APIs. r=shopping-reviewers,jhirsch

Comment 6

[Cosmin Sabou [:CosminS]]

https://hg.mozilla.org/mozilla-central/rev/797c9afc3d0a