Closed Bug 1853299 Opened 1 year ago Closed 1 year ago

Update State to include Analysis Status

Categories

(Fenix :: Shopping, task)

All
Android
task

Tracking

(firefox119 fixed)

RESOLVED FIXED
119 Branch
Tracking Status
firefox119 --- fixed

People

(Reporter: rsainani, Assigned: rsainani)

References

Details

(Whiteboard: [fxdroid] [fakespot-android-foxfooding] [fakespot-android-mvp])

Attachments

(2 files)

This involves using Analysis Status to update the State for success and error cases:

  1. Update AnalysisPresent to include states for stale review, reaanalyzing, up to date and complete as only one of them will be shown. Currently it only supports the state for showing stale review.
  2. Update Error case to include "not_analyzable", "unprocessable", "not_found" states from analysis_status. Also include a state for network error and a generic error.

Note: Based on the docs, Unprocessable shouldn't happen on mobile client, if it does, it's a bug, as we check which websites are supported withisProductUrl`.

No longer depends on: 1840333
Whiteboard: [fxdroid] → [fxdroid] [fakespot-android-foxfooding] [fakespot-android-mvp]
Assignee: nobody → rsainani
Status: NEW → ASSIGNED
Status: ASSIGNED → RESOLVED
Closed: 1 year ago
Flags: qe-verify+
Resolution: --- → FIXED
Target Milestone: --- → 119 Branch
Status: RESOLVED → REOPENED
Flags: qe-verify+
Resolution: FIXED → ---
Status: REOPENED → ASSIGNED
Status: ASSIGNED → RESOLVED
Closed: 1 year ago1 year ago
Flags: qe-verify+
Resolution: --- → FIXED
Flags: qe-verify+
Blocks: 1855921
No longer blocks: 1855921
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: