Skip to content
Snippets Groups Projects

Remove dismissal_reason feature flag

Merged Thiago Figueiró requested to merge 398717-feature-flag-cleanup-dismissal_reason into master
All threads resolved!
13 files
+ 14
354
Compare changes
  • Side-by-side
  • Inline
Files
13
@@ -81,7 +81,7 @@ export default {
return this.selectedStatus === 'dismissed';
},
needsDismissalReasonSelection() {
return this.glFeatures.dismissalReason && this.isDismissedStatus;
return this.isDismissedStatus;
},
canChange() {
if (this.needsDismissalReasonSelection) {
@@ -112,15 +112,10 @@ export default {
let fulfilledCount = 0;
const promises = this.selectedVulnerabilities.map((vulnerability) => {
let variables;
if (this.glFeatures.dismissalReason) {
variables = { id: vulnerability.id, comment: this.comment };
const variables = { id: vulnerability.id, comment: this.comment };
if (this.selectedDismissalReason) {
variables.dismissalReason = this.selectedDismissalReason.toUpperCase();
}
} else {
variables = { id: vulnerability.id, ...this.selectedStatusObject.payload };
if (this.selectedDismissalReason) {
variables.dismissalReason = this.selectedDismissalReason.toUpperCase();
}
return this.$apollo
@@ -257,7 +252,7 @@ export default {
/>
<gl-form-input
v-if="selectedStatus && glFeatures.dismissalReason"
v-if="selectedStatus"
v-model="comment"
:placeholder="commentPlaceholder"
:disabled="isSubmitting"
Loading