Skip to content

Commit b49243d

Browse files
committed
issue #2662, #2430 - In add-links milestone, link value is not cleared when adding the next link
1 parent 7a185cb commit b49243d

File tree

1 file changed

+11
-6
lines changed

1 file changed

+11
-6
lines changed

src/projects/detail/components/timeline/milestones/MilestoneTypeAddLinks/MilestoneTypeAddLinks.jsx

Lines changed: 11 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,7 @@ class MilestoneTypeAddLinks extends React.Component {
2727
this.state = {
2828
addedLinks: [],
2929
isShowCompleteConfirmMessage: false,
30-
isLinkAdded: true,
30+
isLinkUpdating: false,
3131
showExtensionRequestSection: true
3232
}
3333

@@ -87,9 +87,15 @@ class MilestoneTypeAddLinks extends React.Component {
8787
const { addedLinks } = this.state
8888
values.type = 'marvelapp'
8989

90+
// here we simulate uploading process for LinkList component
91+
// as that component is uncontrollable and relies on the `isUpdating` property
9092
this.setState({
91-
addedLinks: [...addedLinks, values],
92-
isLinkAdded: false
93+
isLinkUpdating: true,
94+
}, () => {
95+
this.setState({
96+
addedLinks: [...addedLinks, values],
97+
isLinkUpdating: false,
98+
})
9399
})
94100
}
95101

@@ -101,7 +107,6 @@ class MilestoneTypeAddLinks extends React.Component {
101107
addedLinks.splice(linkIndex, 1)
102108
this.setState({
103109
addedLinks,
104-
isLinkAdded: false
105110
})
106111
}
107112

@@ -117,7 +122,7 @@ class MilestoneTypeAddLinks extends React.Component {
117122
const {
118123
addedLinks,
119124
isShowCompleteConfirmMessage,
120-
isLinkAdded,
125+
isLinkUpdating,
121126
} = this.state
122127

123128
const isActive = milestone.status === MILESTONE_STATUS.ACTIVE
@@ -174,7 +179,7 @@ class MilestoneTypeAddLinks extends React.Component {
174179
formAddButtonTitle="Add link"
175180
formUpdateTitle="Editing a link"
176181
formUpdateButtonTitle="Save changes"
177-
isUpdating={isLinkAdded}
182+
isUpdating={isLinkUpdating}
178183
fakeName={`Design ${addedLinks.length + 1}`}
179184
onFormAddOpen={this.onFormAddOpen}
180185
onFormAddCancel={this.onFormAddCancel}

0 commit comments

Comments
 (0)