Change order of arguments

ui/src/admin/actions/index.js

@@ -146,7 +146,7 @@ export const deleteUserAsync = (user, addFlashMessage) => (dispatch) => {
   deleteUserAJAX(user.links.self, addFlashMessage, user.name)
 }
 
-export const updateRoleUsersAsync = (users, role) => async (dispatch) => {
+export const updateRoleUsersAsync = (role, users) => async (dispatch) => {
   try {
     await updateRoleUsersAJAX(role.links.self, users)
     dispatch(publishNotification('success', 'Role users updated'))
@@ -155,7 +155,7 @@ export const updateRoleUsersAsync = (users, role) => async (dispatch) => {
   }
 }
 
-export const updateRolePermissionsAsync = (permissions, role) => async (dispatch) => {
+export const updateRolePermissionsAsync = (role, permissions) => async (dispatch) => {
   try {
     await updateRolePermissionsAJAX(role.links.self, permissions)
     dispatch(publishNotification('success', 'Role permissions updated'))

ui/src/admin/components/RoleRow.js

@@ -35,11 +35,11 @@ const RoleRow = ({
   onUpdateRolePermissions,
 }) => {
   const handleUpdateUsers = (u) => {
-    onUpdateRoleUsers(u.map((n) => ({name: n})), role)
+    onUpdateRoleUsers(role, u.map((n) => ({name: n})))
   }
 
   const handleUpdatePermissions = (allowed) => {
-    onUpdateRolePermissions([{scope: 'all', allowed}], role)
+    onUpdateRolePermissions(role, [{scope: 'all', allowed}])
   }
 
   const perms = _.get(permissions, ['0', 'allowed'], [])

ui/src/admin/containers/AdminPage.js

@@ -80,12 +80,12 @@ class AdminPage extends Component {
     this.props.deleteUser(user, this.props.addFlashMessage)
   }
 
-  handleUpdateRoleUsers(users, role) {
-    this.props.updateRoleUsers(users, role)
+  handleUpdateRoleUsers(role, users) {
+    this.props.updateRoleUsers(role, users)
   }
 
-  handleUpdateRolePermissions(permissions, role) {
-    this.props.updateRolePermissions(permissions, role)
+  handleUpdateRolePermissions(role, permissions) {
+    this.props.updateRolePermissions(role, permissions)
   }
 
   render() {