Skip to content

Commit 59cb7d7

Browse files
committed
fix: action string
1 parent 537a94b commit 59cb7d7

File tree

2 files changed

+3
-7
lines changed

2 files changed

+3
-7
lines changed

src/components/Users/index.js

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -63,12 +63,8 @@ class Users extends Component {
6363
})
6464
}
6565

66-
resetAddUserState (doReloadList) {
66+
resetAddUserState () {
6767
this.setState({ showAddUserModal: false })
68-
69-
if (doReloadList) {
70-
71-
}
7268
}
7369

7470
resetInviteUserState () {

src/components/Users/user-add.modal.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -93,7 +93,7 @@ const UserAddModalContent = ({
9393
const action = member.role === 'manager' ? 'complete-copilot-requests' : ''
9494
const response = await updateProjectMemberRole(projectId, member.id, 'copilot', action)
9595
updateProjectMember(response)
96-
onClose(true)
96+
onClose()
9797
}
9898

9999
const onCancelCopilotRoleChange = () => {
@@ -104,7 +104,7 @@ const UserAddModalContent = ({
104104
return (
105105
<Modal theme={theme} onCancel={onClose}>
106106
{
107-
isUserAddingFailed && (existingRole === 'observer' || existingRole === 'customer' || existingRole === 'copilot' || existingRole === 'manager') && (
107+
isUserAddingFailed && (['observer', 'customer', 'copilot', 'manager'].includes(existingRole)) && (
108108
<div className={cn(styles.contentContainer, styles.confirm)}>
109109
<div className={styles.textContent}>{`The copilot ${userToAdd.handle} is part of ${projectOption.label} project with ${existingRole} role.`}</div>
110110
<div className={styles.buttonWrapper}>

0 commit comments

Comments
 (0)