UIORGS-436 Settings > Organizations > Banking information is not prop…

…erly fenced off by permissions (#628)
folio-org · Jul 1, 2024 · 19daf93 · 19daf93
1 parent 8d3f8d9
commit 19daf93
Show file tree

Hide file tree

Showing 7 changed files with 66,659 additions and 77,645 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -3,6 +3,7 @@
 ## 5.2.0 (IN PROGRESS)
 
 * UX Consistency: HTML Page Title display when the third pane (detail record) displays. Refs UIORGS-423.
+* Settings > Organizations > Banking information is not properly fenced off by permissions. Refs UIORGS-436.
 
 ## [5.1.1](https://github.com/folio-org/ui-organizations/tree/v5.1.1) (2024-04-18)
 [Full Changelog](https://github.com/folio-org/ui-organizations/compare/v5.1.0...v5.1.1)

diff --git a/...nizationForm/OrganizationAccountsForm/__snapshots__/OrganizationAccountsForm.test.js.snap b/...nizationForm/OrganizationAccountsForm/__snapshots__/OrganizationAccountsForm.test.js.snap
@@ -441,113 +441,109 @@ exports[`OrganizationAccountsForm should render correct structure with defined a
  class="col-xs-6 col-md-3"
  >
  <div
+ aria-expanded="false"
  class="multiSelectContainer"
  id="account-acq-units-0"
+ role="application"
  >
  <div
  aria-live="assertive"
  aria-relevant="additions"
  class="sr-only"
  />
+ <label
+ class="label"
+ for="downshift-:r1:-input"
+ id="multiselect-label-id-account-acq-units-0"
+ >
+ stripes-acq-components.label.acqUnits
+ </label>
+ <span
+ class="sr-only"
+ id="multi-value-status-account-acq-units-0"
+ >
+ 0 items selected
+ </span>
+ <span
+ class="sr-only"
+ id="multi-describe-control-account-acq-units-0"
+ >
+ stripes-components.multiSelection.controlDescription
+ </span>
  <div
- aria-expanded="false"
- aria-haspopup="listbox"
- aria-labelledby="account-acq-units-0-label"
- class="multiSelectContainer"
- role="application"
+ class="multiSelectControlWrapper"
  >
- <label
- class="label"
- for="account-acq-units-0-input"
- id="account-acq-units-0-label"
- >
- stripes-acq-components.label.acqUnits
- </label>
- <span
- class="sr-only"
- id="multi-value-status-account-acq-units-0"
- >
- 0 items selected
- </span>
- <span
- class="sr-only"
- id="multi-describe-control-account-acq-units-0"
- >
- stripes-components.multiSelection.controlDescription
- </span>
  <div
- class="multiSelectControlWrapper"
+ aria-describedby="multi-describe-control-account-acq-units-0"
+ aria-labelledby="multiselect-label-id-account-acq-units-0 multi-value-status-account-acq-units-0"
+ class="multiSelectControl formControl"
+ role="searchbox"
+ tabindex="0"
  >
  <div
- aria-describedby="multi-describe-control-account-acq-units-0"
- aria-labelledby="account-acq-units-0-label multi-value-status-account-acq-units-0"
- class="multiSelectControl formControl"
- role="search"
- tabindex="0"
+ class="multiSelectControlGroup"
+ >
+ <input
+ aria-activedescendant=""
+ aria-autocomplete="list"
+ aria-controls="multiselect-option-list-account-acq-units-0"
+ aria-expanded="false"
+ aria-labelledby="multiselect-label-id-account-acq-units-0"
+ autocomplete="off"
+ class="multiSelectFilterField"
+ id="account-acq-units-0-input"
+ role="combobox"
+ type="text"
+ value=""
+ />
+ <input
+ aria-hidden="true"
+ class="multiSelectValueInput"
+ id="multiselect-input-account-acq-units-0"
+ tabindex="-1"
+ value=""
+ />
+ </div>
+ <button
+ aria-controls="multiselect-option-list-account-acq-units-0"
+ aria-expanded="false"
+ aria-label="toggle menu"
+ class="multiSelectToggleButton"
+ id="downshift-:r1:-toggle-button"
+ tabindex="-1"
+ type="button"
  >
  <div
- class="multiSelectControlGroup"
+ class="textFieldIcon"
  >
- <input
- aria-autocomplete="list"
- aria-labelledby="account-acq-units-0-label"
- autocomplete="off"
- class="multiSelectFilterField"
- id="account-acq-units-0-input"
- type="text"
- value=""
- />
- <input
- aria-autocomplete="list"
- aria-hidden="true"
- autocomplete="off"
- class="multiSelectValueInput"
- id="multiselect-input-account-acq-units-0"
- tabindex="-1"
- type="text"
- value=""
- />
+ <span>
+ Icon
+ </span>
  </div>
- <button
- aria-haspopup="true"
- aria-label="open menu"
- class="multiSelectToggleButton"
- data-toggle="true"
- role="button"
- type="button"
- >
- <div
- class="textFieldIcon"
- >
- <span>
- Icon
- </span>
- </div>
- </button>
- </div>
+ </button>
+ </div>
+ <div
+ class="multiSelectMenu"
+ hidden=""
+ style="width: 100%;"
+ >
  <div
- class="multiSelectMenu"
- hidden=""
- style="width: 100px;"
+ class="multiSelectFeedback"
+ role="alert"
  >
  <div
- class="multiSelectFeedback"
- role="alert"
+ class="multiSelectEmptyMessage"
  >
- <div
- class="multiSelectEmptyMessage"
- >
- stripes-components.multiSelection.defaultEmptyMessage
- </div>
+ stripes-components.multiSelection.defaultEmptyMessage
  </div>
- <ul
- aria-labelledby="account-acq-units-0-label"
- class="multiSelectOptionList"
- id="multiselect-option-list-account-acq-units-0"
- role="listbox"
- style="max-height: 168px;"
- />
  </div>
+ <ul
+ aria-labelledby="multiselect-label-id-account-acq-units-0"
+ class="multiSelectOptionList"
+ id="multiselect-option-list-account-acq-units-0"
+ role="listbox"
+ style="max-height: 168px;"
+ />
  </div>
  </div>
  <div