-
-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
Search: Add option to sort results by picture title #4218
Conversation
Thanks! Let me know when it's fully tested. I'll take a look then and merge it as soon as possible. |
The function itself is already tested in my development environment. By "test case to cover" I mean it probably need to be covered in file such as folder_search_test.go as I didn't check the whole project. Thanks for the quick response! |
I've reviewed the test functions and found nothing else add. The code has passed the unit tests and operates as expected in my test environment. Please proceed with your review of the code. |
Looks good to me! I'll go ahead and merge this for final testing. ✨ |
Signed-off-by: Michael Mayer <michael@photoprism.app>
Signed-off-by: Michael Mayer <michael@photoprism.app>
An updated development preview build is now available for you to test these changes: |
#4214