Skip to content

Commit

Permalink
UIBULKED-377: Not all note types are displayed using "Change note typ…
Browse files Browse the repository at this point in the history
…e" (#408)
  • Loading branch information
UladzislauKutarkin authored Nov 9, 2023
1 parent 7f54e65 commit 488635a
Show file tree
Hide file tree
Showing 4 changed files with 2 additions and 14 deletions.
1 change: 1 addition & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -10,6 +10,7 @@
* [UIBULKED-331](https://issues.folio.org/browse/UIBULKED-331) Bulk edit actions for holdings notes - add and remove notes.
* [UIBULKED-332](https://issues.folio.org/browse/UIBULKED-332) Bulk edit actions for holdings notes - find and replace or remove.
* [UIBULKED-333](https://issues.folio.org/browse/UIBULKED-333) Bulk edit actions for holdings notes - change note type.
* [UIBULKED-377](https://issues.folio.org/browse/UIBULKED-377) Not all note types are displayed using "Change note type".

## [4.0.0](https://github.com/folio-org/ui-bulk-edit/tree/v4.0.0) (2023-10-12)

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -204,17 +204,6 @@ export const ValuesColumn = ({ action, actionIndex, onChange, option }) => {
/>
);

const renderNoteHoldingsTypeSelect = () => controlType === CONTROL_TYPES.HOLDINGS_NOTE && (
<Select
id="noteHoldingsType"
value={action.value}
loading={isHoldingsNotesLoading}
onChange={e => onChange({ actionIndex, value: e.target.value, fieldName: FIELD_VALUE_KEY })}
dataOptions={sortedHoldingsNotes}
aria-label={formatMessage({ id: 'ui-bulk-edit.ariaLabel.loanTypeSelect' })}
/>
);

return (
<Col xs={2} sm={2}>
{renderTextField()}
Expand All @@ -226,7 +215,6 @@ export const ValuesColumn = ({ action, actionIndex, onChange, option }) => {
{renderLoanTypeSelect()}
{renderNoteTypeSelect()}
{renderNoteDuplicateTypeSelect()}
{renderNoteHoldingsTypeSelect()}
</Col>
);
};
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -282,7 +282,7 @@ export const getDefaultActions = (option, options, formatMessage) => {
actionsList: noteWithMarkDefaultActions,
controlType: (action) => {
return action === ACTIONS.CHANGE_TYPE
? CONTROL_TYPES.HOLDINGS_NOTE
? CONTROL_TYPES.NOTE_SELECT
: CONTROL_TYPES.TEXTAREA;
},
[ACTION_VALUE_KEY]: noteWithMarkDefaultActions[0].value,
Expand Down
1 change: 0 additions & 1 deletion src/constants/core.js
Original file line number Diff line number Diff line change
Expand Up @@ -75,7 +75,6 @@ export const CONTROL_TYPES = {
TEXTAREA: 'TEXTAREA',
NOTE_SELECT: 'NOTE_SELECT',
NOTE_DUPLICATE_SELECT: 'NOTE_DUPLICATE_SELECT',
HOLDINGS_NOTE: 'HOLDINGS_NOTE'
};

export const TRANSLATION_SUFFIX = {
Expand Down

0 comments on commit 488635a

Please sign in to comment.