Skip to content

DRYD-2045 Search Results > Authorities > Incorrect default sorted by#344

Merged
spirosdi merged 2 commits intodevelopfrom
bugfix/dryd-2045
Mar 3, 2026
Merged

DRYD-2045 Search Results > Authorities > Incorrect default sorted by#344
spirosdi merged 2 commits intodevelopfrom
bugfix/dryd-2045

Conversation

@spirosdi
Copy link
Contributor

@spirosdi spirosdi commented Mar 3, 2026

What does this do?

  • It adds the defaultSortBy and defaultSortDirection to the columns configuration of authorities. It sets default sorting to termDisplayName:asc
  • It fixes getting defaultSortDirection from config. It also fixes populating sortDirLabel

Why are we doing this? (with JIRA link)
The authorities are sorted by display name: asc, in the backend. However missing configuration leads to showing the default value (updated at: desc) in the Sort By select of the search results page: https://collectionspace.atlassian.net/browse/DRYD-2045

How should this be tested? Do these changes have associated tests?

  1. Run the updates locally
  2. Search for any authority
  3. In the search results, the Sort By select should be set to "Display Name" ascending. In order to confirm the order direction (ascending), you will need to hover the change sort button and check the title/tooltip appearing:
image

Dependencies for merging? Releasing to production?
N/A

Has the application documentation been updated for these changes?
N/A

Did someone actually run this code to verify it works?
@spirosdi ran it locally

Have any new accessibility violations been handled?
N/A

@spirosdi spirosdi requested a review from mikejritter March 3, 2026 18:57
Copy link
Contributor

@mikejritter mikejritter left a comment

Choose a reason for hiding this comment

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

Changes look good. Compared to behavior on core.collectionspace.org and see the same behavior.

@spirosdi spirosdi merged commit 951307f into develop Mar 3, 2026
5 checks passed
@spirosdi spirosdi deleted the bugfix/dryd-2045 branch March 3, 2026 20:33
@spirosdi spirosdi changed the title Bugfix/dryd 2045 DRYD-2045 Search Results > Authorities > Incorrect default sorted by Mar 5, 2026
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