Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: Order Items by weightage in the web items query #26284

Conversation

marination
Copy link
Collaborator

@marination marination commented Jul 1, 2021

Issue:

  • Query engine first queried items in any order, with a page limit and then sorted the results by weightage
  • So items having much higher weightage , which were modified long back, and are ay down the items list don't come up

Fix:

  • order items in query and then again after including website item groups

To Test:

  • Have 10 Items with show in website enabled. Set weightage in each.
  • Make sure the highest weightage item is has the oldest modified time (in the Item List on Desk it should be somewhere at the bottom)
  • Set Products per Page to 5 in Product Settings.
  • Check if this item shows up on the first page of the website product listing

@marination marination merged commit dfe11ac into frappe:version-13-hotfix Jul 6, 2021
@marination marination added the backport develop instructs mergify to frontport PR to develop branch label Jul 6, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport develop instructs mergify to frontport PR to develop branch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant