Skip to content

use sort_title_tsort instead of sort_title_ssort#1228

Merged
akohler merged 1 commit into
mainfrom
DLSR-5/use_title_tsort
Jun 16, 2026
Merged

use sort_title_tsort instead of sort_title_ssort#1228
akohler merged 1 commit into
mainfrom
DLSR-5/use_title_tsort

Conversation

@sourcefilter

@sourcefilter sourcefilter commented Jun 15, 2026

Copy link
Copy Markdown
Member

minor changes:

  • fix a bug where we sorted solr field 'exists' ascending (no value first) instead of descending (records with a value come first)
  • remove very old solr config - current config is in https://github.com/uclalibrary/calursus-solr
  • auto-accept chromatic changes when running after a merge to main

@sourcefilter sourcefilter marked this pull request as draft June 16, 2026 13:34
@sourcefilter sourcefilter force-pushed the DLSR-5/use_title_tsort branch 2 times, most recently from 3392d8d to 8dbe06b Compare June 16, 2026 22:02
minor changes:
- fix a bug where we sorted solr field 'exists' ascending (no value first) instead of descending (records with a value come first)
- remove very old solr config - current config is in https://github.com/uclalibrary/calursus-solr
- auto-accept chromatic changes when running after a merge to main
@sourcefilter sourcefilter force-pushed the DLSR-5/use_title_tsort branch from 8dbe06b to e6786c4 Compare June 16, 2026 22:20
@sourcefilter sourcefilter marked this pull request as ready for review June 16, 2026 22:40
@sourcefilter sourcefilter requested a review from akohler June 16, 2026 22:40

@akohler akohler left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@sourcefilter Looks fine to me, thanks - merging.

@akohler akohler merged commit fb400dc into main Jun 16, 2026
5 checks passed
@akohler akohler deleted the DLSR-5/use_title_tsort branch June 16, 2026 23:49
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.

2 participants