Commit 09c2a929 authored by Kamil Trzciński's avatar Kamil Trzciński

Merge branch '288347-remove-ff-autofix-indicator' into 'master'

Remove feature flag autofix indicator [RUN ALL RSPEC] [RUN AS-IF-FOSS]

See merge request gitlab-org/gitlab!51998
parents f69c6a72 3f4705c1
...@@ -4,15 +4,10 @@ import produce from 'immer'; ...@@ -4,15 +4,10 @@ import produce from 'immer';
import { __ } from '~/locale'; import { __ } from '~/locale';
import securityScannersQuery from '../graphql/queries/project_security_scanners.query.graphql'; import securityScannersQuery from '../graphql/queries/project_security_scanners.query.graphql';
import vulnerabilitiesQuery from '../graphql/queries/project_vulnerabilities.query.graphql'; import vulnerabilitiesQuery from '../graphql/queries/project_vulnerabilities.query.graphql';
import vulnerabilitiesQueryAutoFix from '../graphql/queries/project_vulnerabilities_autofix.query.graphql';
import { preparePageInfo } from '../helpers'; import { preparePageInfo } from '../helpers';
import { VULNERABILITIES_PER_PAGE } from '../store/constants'; import { VULNERABILITIES_PER_PAGE } from '../store/constants';
import VulnerabilityList from './vulnerability_list.vue'; import VulnerabilityList from './vulnerability_list.vue';
const query = gon?.features?.secureVulnerabilityAutofixIndicator
? vulnerabilitiesQueryAutoFix
: vulnerabilitiesQuery;
export default { export default {
name: 'ProjectVulnerabilitiesApp', name: 'ProjectVulnerabilitiesApp',
components: { components: {
...@@ -41,7 +36,7 @@ export default { ...@@ -41,7 +36,7 @@ export default {
}, },
apollo: { apollo: {
vulnerabilities: { vulnerabilities: {
query, query: vulnerabilitiesQuery,
variables() { variables() {
return { return {
fullPath: this.projectFullPath, fullPath: this.projectFullPath,
......
...@@ -23,6 +23,13 @@ query project( ...@@ -23,6 +23,13 @@ query project(
) { ) {
nodes { nodes {
...Vulnerability ...Vulnerability
hasSolutions
mergeRequest {
webUrl
state
securityAutoFix
iid
}
} }
pageInfo { pageInfo {
...PageInfo ...PageInfo
......
#import "~/graphql_shared/fragments/pageInfoCursorsOnly.fragment.graphql"
#import "../fragments/vulnerability.fragment.graphql"
query project(
$fullPath: ID!
$after: String
$first: Int
$severity: [VulnerabilitySeverity!]
$reportType: [VulnerabilityReportType!]
$scanner: [String!]
$state: [VulnerabilityState!]
$sort: VulnerabilitySort
) {
project(fullPath: $fullPath) {
vulnerabilities(
after: $after
first: $first
severity: $severity
reportType: $reportType
scanner: $scanner
state: $state
sort: $sort
) {
nodes {
...Vulnerability
hasSolutions
mergeRequest {
webUrl
state
securityAutoFix
iid
}
}
pageInfo {
...PageInfo
}
}
}
}
...@@ -5,10 +5,6 @@ module Projects ...@@ -5,10 +5,6 @@ module Projects
class VulnerabilityReportController < Projects::ApplicationController class VulnerabilityReportController < Projects::ApplicationController
include SecurityDashboardsPermissions include SecurityDashboardsPermissions
before_action do
push_frontend_feature_flag(:secure_vulnerability_autofix_indicator, @project, default_enabled: true)
end
feature_category :vulnerability_management feature_category :vulnerability_management
alias_method :vulnerable, :project alias_method :vulnerable, :project
......
---
name: secure_vulnerability_autofix_indicator
introduced_by_url: https://gitlab.com/gitlab-org/gitlab/-/merge_requests/48251/
rollout_issue_url: https://gitlab.com/gitlab-org/gitlab/-/issues/288347
milestone: '13.7'
type: development
group: group::composition analysis
default_enabled: true
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment