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.

Sign up for GitHub

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

Jump to bottom

🐛 Fix "Sort by: Published Date" #2316

Merged
merged 2 commits into from
Sep 10, 2024
Merged

🐛 Fix "Sort by: Published Date" #2316

merged 2 commits into from
Sep 10, 2024

Commits on Sep 9, 2024

  1. 🐛 Fix "Sort by: Published Date" 

    This commit fixes a bug in the ability to sort works by "Published
date", "Upload date", and "Title" in the catalog search page and the
collection show page.

Ref:
- scientist-softserv/adventist_knapsack#185
    @sjproctor
    sjproctor committed Sep 9, 2024
    Configuration menu
    Copy the full SHA
    2a7de90 View commit details
    Browse the repository at this point in the history

  2. ✅ Adds a guard clause to fix failing tests 

    This commit adds a guard clause to the `SortTitle` model to fix failing
tests due to a nil value being assigned to `@title` on initialization.

Ref:
- scientist-softserv/adventist_knapsack#185
    @sjproctor
    sjproctor committed Sep 9, 2024
    Configuration menu
    Copy the full SHA
    2846f44 View commit details
    Browse the repository at this point in the history