Skip to content

Commit

Permalink
fix: Remove sort options from ISSN(Print) and ISSN(Electronic) column…
Browse files Browse the repository at this point in the history
…s on Journal sort

We do not currently support sorting journal list by ISSN, so neither ISSN (Print) no ISSN (Electronic) columns should not appear as a sortable column in the UI
  • Loading branch information
Jack-Golding committed Oct 9, 2024
1 parent 39f1686 commit 5d3096a
Showing 1 changed file with 9 additions and 3 deletions.
12 changes: 9 additions & 3 deletions src/routes/JournalsRoute/JournalsRoute.js
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,10 @@ import {
} from '@folio/stripes/components';
import { SASQRoute } from '@k-int/stripes-kint-components';

import { OAFilterHeaderComponent, JournalsFilters } from '../../components/SearchAndFilter';
import {
OAFilterHeaderComponent,
JournalsFilters,
} from '../../components/SearchAndFilter';
import { findIssnByNamespace } from '../../util/journalUtils';
import Journal from '../../components/views/Journal';
import { JournalModal } from '../../components/Modals';
Expand All @@ -23,7 +26,7 @@ import { WORKS_ENDPOINT } from '../../constants/endpoints';
import {
MAIN_FILTER_PANE_CONFIG,
MAIN_PANE_ID,
MAIN_PANESET_CONFIG
MAIN_PANESET_CONFIG,
} from '../../constants/panesetConfigs';

const JournalsRoute = ({ path }) => {
Expand Down Expand Up @@ -131,7 +134,10 @@ const JournalsRoute = ({ path }) => {
path={path}
persistedPanesetProps={MAIN_PANESET_CONFIG}
resultColumns={resultColumns}
sasqProps={{ initialSortState: { sort: 'title' } }}
sasqProps={{
initialSortState: { sort: 'title' },
sortableColumns: ['title'],
}}
searchFieldAriaLabel="journals-search-field"
ViewComponent={Journal}
/>
Expand Down

0 comments on commit 5d3096a

Please sign in to comment.