Commit b91638c5 authored by Kushal Pandya's avatar Kushal Pandya

Merge branch 'remove-pipeline-summary-report' into 'master'

Remove Feature flag for Pipelines Security Summary [RUN ALL RSPEC] [RUN AS-IF-FOSS]

See merge request gitlab-org/gitlab!54808
parents 98ef439c c597dcc0
...@@ -13,7 +13,6 @@ class Projects::PipelinesController < Projects::ApplicationController ...@@ -13,7 +13,6 @@ class Projects::PipelinesController < Projects::ApplicationController
before_action :authorize_create_pipeline!, only: [:new, :create, :config_variables] before_action :authorize_create_pipeline!, only: [:new, :create, :config_variables]
before_action :authorize_update_pipeline!, only: [:retry, :cancel] before_action :authorize_update_pipeline!, only: [:retry, :cancel]
before_action do before_action do
push_frontend_feature_flag(:pipelines_security_report_summary, project, default_enabled: :yaml)
push_frontend_feature_flag(:new_pipeline_form, project, default_enabled: :yaml) push_frontend_feature_flag(:new_pipeline_form, project, default_enabled: :yaml)
push_frontend_feature_flag(:graphql_pipeline_details, project, type: :development, default_enabled: :yaml) push_frontend_feature_flag(:graphql_pipeline_details, project, type: :development, default_enabled: :yaml)
push_frontend_feature_flag(:graphql_pipeline_details_users, current_user, type: :development, default_enabled: :yaml) push_frontend_feature_flag(:graphql_pipeline_details_users, current_user, type: :development, default_enabled: :yaml)
......
---
name: pipelines_security_report_summary
introduced_by_url: https://gitlab.com/gitlab-org/gitlab/-/merge_requests/31136
rollout_issue_url: https://gitlab.com/gitlab-org/gitlab/-/issues/235943
milestone: '13.0'
type: development
group: group::dynamic analysis
default_enabled: true
...@@ -3,7 +3,6 @@ import { GlEmptyState } from '@gitlab/ui'; ...@@ -3,7 +3,6 @@ import { GlEmptyState } from '@gitlab/ui';
import { mapActions } from 'vuex'; import { mapActions } from 'vuex';
import { fetchPolicies } from '~/lib/graphql'; import { fetchPolicies } from '~/lib/graphql';
import { s__ } from '~/locale'; import { s__ } from '~/locale';
import glFeatureFlagsMixin from '~/vue_shared/mixins/gl_feature_flags_mixin';
import pipelineSecurityReportSummaryQuery from '../graphql/queries/pipeline_security_report_summary.query.graphql'; import pipelineSecurityReportSummaryQuery from '../graphql/queries/pipeline_security_report_summary.query.graphql';
import SecurityDashboard from './security_dashboard_vuex.vue'; import SecurityDashboard from './security_dashboard_vuex.vue';
import SecurityReportsSummary from './security_reports_summary.vue'; import SecurityReportsSummary from './security_reports_summary.vue';
...@@ -15,7 +14,6 @@ export default { ...@@ -15,7 +14,6 @@ export default {
SecurityReportsSummary, SecurityReportsSummary,
SecurityDashboard, SecurityDashboard,
}, },
mixins: [glFeatureFlagsMixin()],
apollo: { apollo: {
securityReportSummary: { securityReportSummary: {
query: pipelineSecurityReportSummaryQuery, query: pipelineSecurityReportSummaryQuery,
...@@ -30,9 +28,6 @@ export default { ...@@ -30,9 +28,6 @@ export default {
const summary = data?.project?.pipeline?.securityReportSummary; const summary = data?.project?.pipeline?.securityReportSummary;
return summary && Object.keys(summary).length ? summary : null; return summary && Object.keys(summary).length ? summary : null;
}, },
skip() {
return !this.glFeatures.pipelinesSecurityReportSummary;
},
}, },
}, },
props: { props: {
......
---
title: Remove feature flag for Pipelines Security Summary
merge_request: 54808
author:
type: other
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