[MI-2814] Done the review fixes of github PR #636

webapp/src/components/modals/attach_comment_to_issue/attach_comment_to_issue.jsx

@@ -72,7 +72,7 @@ export default class AttachIssueModal extends PureComponent {
             owner,
             repo,
             number,
-            comment: this.props.post.message,
+            comment: this.state.comment,
             post_id: this.props.post.id,
             show_attached_message: true,
         };
@@ -106,9 +106,17 @@ export default class AttachIssueModal extends PureComponent {
         });
     };
 
+    /* eslint-disable react/no-did-update-set-state*/
+    componentDidUpdate(prevProps) {
+        if (this.props.post && !this.props.messageData && !prevProps.post) {
+            this.setState({comment: this.props.post.message});
+        }
+    }
+    /* eslint-enable */
+
     render() {
         const {error, submitting, comment, issueValue} = this.state;
-        const {visible, theme, messageData, post} = this.props;
+        const {visible, theme, messageData} = this.props;
         const style = getStyle(theme);
         if (!visible) {
             return null;
@@ -138,10 +146,9 @@ export default class AttachIssueModal extends PureComponent {
                 <Input
                     label='Message Attached to GitHub Issue'
                     type='textarea'
-                    isDisabled={false}
-                    value={post?.message}
+                    value={comment}
                     disabled={false}
-                    readOnly={true}
+                    onChange={this.handleIssueCommentChange}
                 />
             </div>
         );

webapp/src/components/modals/create_update_issue/create_update_issue.jsx

@@ -22,7 +22,7 @@ const initialState = {
     repo: null,
     issueTitle: '',
     issueDescription: '',
-    channelID: '',
+    channelId: '',
     labels: [],
     assignees: [],
     milestone: null,
@@ -62,23 +62,23 @@ export default class CreateOrUpdateIssueModal extends PureComponent {
             labels = [];
         }
 
-        this.setState({assignees});
-        this.setState({labels});
         this.setState({milestone: {
             value: milestone_number,
             label: milestone_title,
         },
         repo: {
             name: repo_full_name,
         },
-        channelID: channel_id,
+        assignees,
+        labels,
+        channelId: channel_id,
         issueDescription: description,
         issueTitle: title.substring(0, MAX_TITLE_LENGTH)});
     }
 
     /* eslint-disable react/no-did-update-set-state*/
     componentDidUpdate(prevProps) {
-        if (this.props.post && !this.props.messageData) {
+        if (this.props.post && !this.props.messageData && !prevProps.post) {
             this.setState({issueDescription: this.props.post.message});
         }
 
@@ -118,7 +118,7 @@ export default class CreateOrUpdateIssueModal extends PureComponent {
             assignees: this.state.assignees,
             milestone: this.state.milestone && this.state.milestone.value,
             post_id: postId,
-            channel_id: this.state.channelID,
+            channel_id: this.state.channelId,
             issue_number,
         };
 
@@ -252,6 +252,7 @@ export default class CreateOrUpdateIssueModal extends PureComponent {
                     onChange={this.handleIssueTitleChange}
                 />
                 {issueTitleValidationError}
+
                 {this.renderIssueAttributeSelectors()}
 
                 <Input