Fixes #36368 - fix ansible roles pagination on host edit page

(cherry picked from commit 2a95285)
theforeman · Aug 9, 2023 · 28756ee · 28756ee
1 parent 901016e
commit 28756ee
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 10 deletions.
diff --git a/webpack/components/AnsibleRolesSwitcher/AnsibleRolesSwitcher.js b/webpack/components/AnsibleRolesSwitcher/AnsibleRolesSwitcher.js
@@ -26,7 +26,6 @@ class AnsibleRolesSwitcher extends React.Component {
       inheritedRoleIds,
       resourceId,
       resourceName,
-      { page: 1, perPage: 10 },
       excludeAssignedRolesSearch(initialAssignedRoles)
     );
   }

diff --git a/webpack/components/AnsibleRolesSwitcher/components/AvailableRolesList.js b/webpack/components/AnsibleRolesSwitcher/components/AvailableRolesList.js
@@ -19,7 +19,9 @@ const AvailableRolesList = ({
       <Pagination
         viewType="list"
         itemCount={itemCount}
-        pagination={pagination}
+        updateParamsByUrl={false}
+        page={pagination.page}
+        perPage={pagination.perPage}
         onChange={onListingChange}
         dropdownButtonId="available-ansible-roles-pagination-row-dropdown"
       />

diff --git a/.../components/AnsibleRolesSwitcher/components/__snapshots__/AvailableRolesList.test.js.snap b/.../components/AnsibleRolesSwitcher/components/__snapshots__/AvailableRolesList.test.js.snap
@@ -16,14 +16,8 @@ exports[`AvailableRolesList should render 1`] = `
       onPerPageSelect={null}
       onSetPage={null}
       page={1}
-      pagination={
-        Object {
-          "page": 1,
-          "perPage": 25,
-        }
-      }
-      perPage={null}
-      updateParamsByUrl={true}
+      perPage={25}
+      updateParamsByUrl={false}
       variant="bottom"
       viewType="list"
     />