Update State to include Analysis Status
Categories
(Fenix :: Shopping, task)
Tracking
(firefox119 fixed)
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:
- 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. - 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 with
isProductUrl`.
Updated•1 year ago
|
Assignee | ||
Updated•1 year ago
|
Assignee | ||
Updated•1 year ago
|
Comment 1•1 year ago
|
||
Comment 2•1 year ago
|
||
Authored by https://github.com/rahulsainani
https://github.com/mozilla-mobile/firefox-android/commit/54ded217bee4a33b01369f4e40b5469bd86ebb13
[main] Bug 1853299 - Part 1: Prepare rc state for reanalysis
Authored by https://github.com/rahulsainani
https://github.com/mozilla-mobile/firefox-android/commit/fd348c329caaa94d3ffb5ef244fb1c2660677538
[main] Bug 1853299 - Part 2: Update rc non error states for reanalysis
Assignee | ||
Updated•1 year ago
|
Assignee | ||
Updated•1 year ago
|
Comment 3•1 year ago
|
||
Comment 4•1 year ago
|
||
Authored by https://github.com/rahulsainani
https://github.com/mozilla-mobile/firefox-android/commit/576e02373e7c0400664f44f54aa0e63b59da0aa2
[main] Bug 1853299 - Part 3: Update review checker error states
Assignee | ||
Updated•1 year ago
|
Description
•