Skip to content

Commit b4443ff

Browse files
committed
fix join project roles
1 parent 9f5988e commit b4443ff

File tree

3 files changed

+4
-4
lines changed

3 files changed

+4
-4
lines changed

src/components/TeamManagement/TeamManagement.jsx

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -159,8 +159,8 @@ class TeamManagement extends React.Component {
159159
</div>
160160
{isShowJoin && ((() => {
161161
const onClickCancel = () => onJoin(false)
162-
const onClickJoinConfirm = () => {
163-
onJoinConfirm()
162+
const onClickJoinConfirm = (role) => {
163+
onJoinConfirm(role)
164164
}
165165
return (
166166
<Dialog

src/projects/actions/projectMember.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -34,7 +34,7 @@ function addProjectMemberWithData(dispatch, projectId, member) {
3434
return new Promise((resolve, reject) => {
3535
return dispatch({
3636
type: ADD_PROJECT_MEMBER,
37-
payload: addMember(projectId, member)
37+
payload: addMember(projectId, {role: member.role})
3838
})
3939
.then((/*{value, action}*/) => {
4040
return resolve(dispatch(loadMembers([member.userId])))

src/projects/detail/containers/TeamManagementContainer.jsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -71,7 +71,7 @@ class TeamManagementContainer extends Component {
7171

7272
onJoinConfirm(role) {
7373
const { currentUser, projectId, addProjectMember } = this.props
74-
role = role || currentUser.isCopilot ? PROJECT_ROLE_COPILOT : PROJECT_ROLE_MANAGER
74+
role = role || (currentUser.isCopilot ? PROJECT_ROLE_COPILOT : PROJECT_ROLE_MANAGER)
7575
addProjectMember(
7676
projectId,
7777
{userId: currentUser.userId, role}

0 commit comments

Comments
 (0)