Skip to content

Search: Add option to sort results by picture title#4218

Merged
lastzero merged 1 commit intophotoprism:developfrom
wsyxbcl:title-sort
May 18, 2024
Merged

Search: Add option to sort results by picture title#4218
lastzero merged 1 commit intophotoprism:developfrom
wsyxbcl:title-sort

Conversation

@wsyxbcl
Copy link
Contributor

@wsyxbcl wsyxbcl commented Apr 26, 2024

@CLAassistant
Copy link

CLAassistant commented Apr 26, 2024

CLA assistant check
All committers have signed the CLA.

@lastzero
Copy link
Member

Thanks! Let me know when it's fully tested. I'll take a look then and merge it as soon as possible.

@wsyxbcl
Copy link
Contributor Author

wsyxbcl commented Apr 26, 2024

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!

@wsyxbcl
Copy link
Contributor Author

wsyxbcl commented May 6, 2024

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.

@lastzero
Copy link
Member

Looks good to me! I'll go ahead and merge this for final testing. ✨

@lastzero lastzero merged commit 7e18a41 into photoprism:develop May 18, 2024
lastzero added a commit that referenced this pull request May 18, 2024
Signed-off-by: Michael Mayer <michael@photoprism.app>
lastzero added a commit that referenced this pull request May 18, 2024
Signed-off-by: Michael Mayer <michael@photoprism.app>
@lastzero lastzero added enhancement Enhancement or improvement of an existing feature please-test Ready for acceptance test merged Changes are merged, but may require further testing labels May 18, 2024
@lastzero lastzero changed the title Search: Add sort by Title Search: Add option to sort by picture title May 18, 2024
@lastzero lastzero changed the title Search: Add option to sort by picture title Search: Add option to sort results by picture title May 18, 2024
@lastzero
Copy link
Member

An updated development preview build is now available for you to test these changes:

@graciousgrey graciousgrey added tested Changes have been tested successfully released Available in the stable release and removed please-test Ready for acceptance test tested Changes have been tested successfully labels May 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement Enhancement or improvement of an existing feature merged Changes are merged, but may require further testing released Available in the stable release

Projects

Status: Release 🌈

Development

Successfully merging this pull request may close these issues.

4 participants