Closed Bug 1855974 Opened 9 months ago Closed 8 months ago

The Unsupported product error message is not displayed for unsupported products in the Shopping sidebar

Categories

(Firefox :: Shopping, defect)

Desktop
All
defect

Tracking

()

RESOLVED FIXED
121 Branch
Tracking Status
firefox-esr115 --- disabled
firefox118 --- disabled
firefox119 --- disabled
firefox120 --- verified
firefox121 --- verified

People

(Reporter: rdoghi, Assigned: fchasen)

References

(Blocks 1 open bug)

Details

(Whiteboard: [fidefe-shopping])

Attachments

(1 file)

Found in

  • Beta 119.0b1

Affected versions

  • Beta 119.0b1
  • Nightly 120.0a1 (2023-09-29)

Affected platforms

  • All

Preconditions:
Set the browser.shopping.experience2023.enabled - TRUE
Set the browser.shopping.experience2023.optedIn - 1

Steps to reproduce

  1. Reach the following links:
    BestBuy gift card: https://www.bestbuy.com/site/best-buy-50-best-buy-gift-card-with-gift-box/6349639.p?skuId=6349639
    VideoGames: https://www.amazon.com/luna/detail/B0C3QB7QKV
    Amazon music: https://www.amazon.com/dp/B0C1Q4GWXJ/
  2. Open the Sidebar.

Expected result

  • The Unsupported product error message should be displayed for these products.

Actual result

  • In some cases the Check review quality card is displayed again in infinite loop, while in other cases the error message is not displayed instead the reviews are being displayed.

Regression range
Not Applicable

Assignee: nobody → fchasen
Status: NEW → ASSIGNED
  • Throws errors when we get an response that there was an issue from the Shopping Status and Analyze APIs.
  • Updates the ShoppingContainer to handle error states that have non-generic messages to show.

This should hopefully get fixed on Monday when server starts returning a correct flag for these products - https://mozilla-hub.atlassian.net/browse/FAK-501

Let's please reverify it on Tuesday to make sure that unsupported products act as expected and return a correct error when attempting to analyze them
Here's an example of a product where we receive the flag correctly - https://www.amazon.com/Amazon-com-Greeting-Birthday-Birds-Design/dp/B0773DDXGL/?th=1

Attachment #9357194 - Attachment description: WIP: Bug 1855974 - Error when Shopping Status API status returns issues with an analysis. → WIP: Bug 1855974 - Use Shopping Status API status for data.
Attachment #9357194 - Attachment description: WIP: Bug 1855974 - Use Shopping Status API status for data. → WIP: Bug 1855974 - Use Shopping Status API status for data or throw on error values.

Ok, so the correct dependency here is this server-side ticket - https://mozilla-hub.atlassian.net/browse/FAK-570.

Attachment #9357194 - Attachment description: WIP: Bug 1855974 - Use Shopping Status API status for data or throw on error values. → Bug 1855974 - Use Shopping Status API status as data or errors. r=#shopping-reviewers
Pushed by fchasen@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/277291259820
Use Shopping Status API status as data or errors. r=shopping-reviewers,Gijs
Status: ASSIGNED → RESOLVED
Closed: 8 months ago
Resolution: --- → FIXED
Target Milestone: --- → 121 Branch

The patch landed in nightly and beta is affected.
:fchasen, is this bug important enough to require an uplift?

  • If yes, please nominate the patch for beta approval.
  • If no, please set status-firefox120 to wontfix.

For more information, please visit BugBot documentation.

Flags: needinfo?(fchasen)

Hi @fchasen, it seems that for the https://www.bestbuy.com/site/best-buy-50-best-buy-gift-card-with-gift-box/6349639.p?skuId=6349639 product we still see the reviews instead of the Unsupported message can you please take a look ?

Thanks @rares.doghi, unfortunately we are dependent on the Fakespot API to mark products as unsupported.

Maybe we can come up with a way to file bugs with them for specific products that aren't working correctly? Slack seems to work ok for now though.

Flags: needinfo?(fchasen) → needinfo?(rdoghi)

@rares.doghi and @fchasen: Fakespot is working on fixing this bug.

Marking this issue Verified as fixed since the issue with the gift cards is being fixed on the Server side by the Fakespot team.

Flags: needinfo?(rdoghi)

Is this working for 120 as well then?

Flags: needinfo?(rdoghi)

Yes, the issue seems to be fixed in 120.0b7. Should we update the Flag from affected to Verified ?

Flags: needinfo?(rdoghi)
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: