Closed Bug 1862403 Opened 1 year ago Closed 1 year ago

Remove unneeded states from RecommendedProductState like Loading and Error.

Categories

(Fenix :: Shopping, task, P2)

All
Android
task

Tracking

(firefox121 verified)

VERIFIED FIXED
121 Branch
Tracking Status
firefox121 --- verified

People

(Reporter: rsainani, Assigned: rebecatudor273)

References

(Blocks 1 open bug)

Details

(Whiteboard: [fxdroid])

Attachments

(1 file)

Currently it contains 4 states - Initial, Loading, Error and Product. This is correct however the UI only uses 2 states and doesn't plan to use the other states. It would be ideal to have only 2 states or use nullable Product to model the RecommendedProductState.

This also has the good side effect of reducing recompositions since there are no unwanted state changes.

Severity: -- → N/A
Priority: -- → P2
Assignee: nobody → rebecatudor273
Whiteboard: [fxdroid]
Status: NEW → RESOLVED
Closed: 1 year ago
Flags: qe-verify+
Resolution: --- → FIXED
Target Milestone: --- → 121 Branch

After these changes the behavior shouldn't be changed.

The expected behavior:
Setup:

  • Shopping experience is enabled
  • Review Checker is opted in
  • The setting from Review Checker for Product Recommendation is enabled

When you are on a product page from a supported retailers like this, tap on the tags and in Review Checker, a product should be showed.

Verified on the latest Nightly 121.0a1 with Google Pixel 6 (Android 14), and Oppo Find N2 Flip (Android 13).

I've tested this, by enabling the Android Nightly experiment (in PREVIEW mode only) for Fakespot ads: https://experimenter.services.mozilla.com/nimbus/android-review-checker-smoke-test-with-promos/summary, by using Nightly's secret settings to manually enroll in this preview experiment.

I've turned ON the "Show ads in review checker" switch, reload the shopping product page (standing desk product), and observed the "More to consider" area that displayed a link to a similar product/recommended prodyct. I've tapped the link was redirected to the recommended product page.

Status: RESOLVED → VERIFIED
Flags: qe-verify+
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: