Skip to content

Conversation

@tom-anders
Copy link
Collaborator

@tom-anders
Copy link
Collaborator Author

Leaving this as a draft for now because:

  • Implement study search sorting lila#18427 is not deployed to production server yet
  • It's deployed to lichess.dev, but study search is disabled there
  • My PC can't handle a local lila instance with search enabled :D

@ijm8710
Copy link

ijm8710 commented Oct 31, 2025

Leaving this as a draft for now because:

  • Implement study search sorting lila#18427 is not deployed to production server yet

  • It's deployed to lichess.dev, but study search is disabled there

  • My PC can't handle a local lila instance with search enabled :D

Hey tom, I "think" it's deployed to prod now.

Having said that, as a user who was waiting/excited specifically for study search to improved, I fear the implementation is a bit disappointing (not sure if you agree)
Not saying a ton of investment needs to go in, but it seems to do a pretty poor job of surfacing relevant matches, if you choose by "most popular" that doesn't necessarily mean it's a good way to match it.
Curious if you agree

@lenguyenthanh
Copy link
Member

Having said that, as a user who was waiting/excited specifically for study search to improved, I fear the implementation is a bit disappointing (not sure if you agree)
Not saying a ton of investment needs to go in, but it seems to do a pretty poor job of surfacing relevant matches, if you choose by "most popular" that doesn't necessarily mean it's a good way to match it.

Hey, I heard you, I did some more improvement with study search, do you mind to take a look and give some more feedback 🙏

@tom-anders tom-anders marked this pull request as ready for review November 30, 2025 13:16
@tom-anders
Copy link
Collaborator Author

Server-side change is deployed to lichess.org, so this would be ready for review now.

@veloce I don't have the time right now to add a video showcasing this, but I can do it in the following days if you want. Should be pretty easy to test it yourself otherwise

@tom-anders tom-anders closed this Nov 30, 2025
@tom-anders tom-anders reopened this Nov 30, 2025
@veloce veloce merged commit bbd4f98 into lichess-org:main Dec 1, 2025
2 checks passed
@veloce
Copy link
Contributor

veloce commented Dec 1, 2025

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Study sort option not working on the Beta app.

4 participants