Skip to content
This repository has been archived by the owner on Feb 23, 2024. It is now read-only.

Product Query: [FSE] Sort order displayed incorrectly on the front-end #7626

Closed
Tracked by #42616
sunyatasattva opened this issue Nov 8, 2022 · 3 comments
Closed
Tracked by #42616
Labels
block-type: product-query Issues related to/affecting all product-query variations. priority: high The issue/PR is high priority—it affects lots of customers substantially, but not critically. status: on hold The issue is currently not prioritized or is awaiting something (for example, a fix upsteam). type: bug The issue/PR concerns a confirmed bug.

Comments

@sunyatasattva
Copy link
Contributor

sunyatasattva commented Nov 8, 2022

⚠️ Warning
In order to reproduce this bug, #7382 is required

Describe the bug

The sort order seems to be incorrect on the SSR block on the front-end. This can be easily seen as when a classic template block is set to “Sort by latest”, the order in which products are displayed on the front-end differs. Meanwhile, the sort order seems to coincide in the editor.

To reproduce

Steps to reproduce the behavior:

  1. Go to the “Product Catalog” template.
  2. Add a “Product Query” block, with “Order by”: “Newest to oldest”.
  3. Compare the results in the preview with the results in the front-end and the classic template.

Screenshots

Screenshot 2022-11-08 at 23 52 05

Screenshot 2022-11-08 at 23 51 57

@sunyatasattva sunyatasattva added type: bug The issue/PR concerns a confirmed bug. block-type: product-query Issues related to/affecting all product-query variations. labels Nov 8, 2022
@sunyatasattva sunyatasattva changed the title Product Query: Sort order displayed incorrectly on the front-end Product Query: [FSE] Sort order displayed incorrectly on the front-end Nov 9, 2022
@sunyatasattva sunyatasattva added the status: on hold The issue is currently not prioritized or is awaiting something (for example, a fix upsteam). label Dec 13, 2022
@imanish003 imanish003 added the priority: high The issue/PR is high priority—it affects lots of customers substantially, but not critically. label Apr 18, 2023
@imanish003
Copy link
Contributor

Hi @gigitux, I attempted to reproduce this issue, but it isn't happening for me. 🤷🏻‍♂️ Additionally, it appears that this is within the 'Only required with custom "Inherit from template" logic' section of the Products block Epic woocommerce/woocommerce#42616. Perhaps we can close this issue for now?

image

@kmanijak
Copy link
Contributor

kmanijak commented Apr 19, 2023

I could reproduce, but with the custom "Inherit from template" logic.

I added the notification at the top of the PR description, the same as in #7608.

Perhaps we can close this issue for now?

I think we could approach this issue the same as #7608, so I'd close it for now, and if we decide to follow the custom logic, then we can reopen.

@imanish003
Copy link
Contributor

That makes sense to me. I'll close it for now. Thank you, Karol, for adding the notification at the top of the PR description. 🙌🏻

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
block-type: product-query Issues related to/affecting all product-query variations. priority: high The issue/PR is high priority—it affects lots of customers substantially, but not critically. status: on hold The issue is currently not prioritized or is awaiting something (for example, a fix upsteam). type: bug The issue/PR concerns a confirmed bug.
Projects
None yet
Development

No branches or pull requests

3 participants