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

Add label support to HiddenSorting options for scheduled searches. #4252

Merged
merged 5 commits into from
Feb 18, 2025

Conversation

EreMaijala
Copy link
Contributor

@EreMaijala EreMaijala commented Feb 17, 2025

Adds support for defining labels for HiddenSorting patterns so that they can be properly displayed in the sort option selection. Also adds a default entry for search links in scheduled search notifications.

TODO

  • Update changelog when merging (note changed format of hidden sort options)

Copy link
Member

@demiankatz demiankatz left a comment

Choose a reason for hiding this comment

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

Thanks, @EreMaijala -- just a couple of small suggestions.

@demiankatz demiankatz added this to the 11.0 milestone Feb 18, 2025
Copy link
Member

@demiankatz demiankatz left a comment

Choose a reason for hiding this comment

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

Thanks again, @EreMaijala!

@demiankatz demiankatz merged commit fcd6722 into vufind-org:dev Feb 18, 2025
6 checks passed
@demiankatz demiankatz deleted the dev-hiddensorting-labels branch February 18, 2025 13:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants