Skip to content

Commit eaf1abd

Browse files
committed
Fix initial reviewer field and do not re-post resource to server
1 parent b9ae916 commit eaf1abd

File tree

1 file changed

+5
-2
lines changed

1 file changed

+5
-2
lines changed

src/components/ChallengeEditor/index.js

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -155,7 +155,9 @@ class ChallengeEditor extends Component {
155155
try {
156156
const copilotResource = this.getResourceFromProps('Copilot')
157157
const copilotFromResources = copilotResource ? copilotResource.memberHandle : ''
158-
const reviewerResource = this.getResourceFromProps('Reviewer')
158+
const reviewerResource =
159+
(challengeDetails.type === 'First2Finish' || challengeDetails.type === 'Task')
160+
? this.getResourceFromProps('Iterative Reviewer') : this.getResourceFromProps('Reviewer')
159161
const reviewerFromResources = reviewerResource ? reviewerResource.memberHandle : ''
160162
setState({ isConfirm: false, isLaunch: false })
161163
const challengeData = this.updateAttachmentlist(challengeDetails, attachments)
@@ -966,7 +968,8 @@ class ChallengeEditor extends Component {
966968
const { copilot: previousCopilot, reviewer: previousReviewer } = this.state.draftChallenge.data
967969
if (copilot !== previousCopilot) await this.updateResource(challengeId, 'Copilot', copilot, previousCopilot)
968970
if (type === 'First2Finish' || type === 'Task') {
969-
await this.updateResource(challengeId, 'Iterative Reviewer', reviewer)
971+
const { memberHandle: previousIterativeReviewer } = this.getResourceFromProps('Iterative Reviewer')
972+
if (reviewer !== previousIterativeReviewer) await this.updateResource(challengeId, 'Iterative Reviewer', reviewer, previousIterativeReviewer)
970973
} else {
971974
if (reviewer !== previousReviewer) await this.updateResource(challengeId, 'Reviewer', reviewer, previousReviewer)
972975
}

0 commit comments

Comments
 (0)