Skip to content
Snippets Groups Projects
Commit 3c9977bc authored by Bob Van Landuyt :neckbeard:'s avatar Bob Van Landuyt :neckbeard: :sunglasses:
Browse files

Merge branch 'pl-remove-feature-flag-status-page-attachments' into 'master'

Remove feature flag `status_page_attachments`

Closes #219286

See merge request gitlab-org/gitlab!42394
parents bfc4f278 4b07646e
No related branches found
No related tags found
No related merge requests found
Loading
Loading
@@ -40,8 +40,6 @@ module StatusPage
end
 
def publish_attachments(issue, user_notes)
return success unless attachements_enabled?
StatusPage::PublishAttachmentsService.new(
project: @project,
issue: issue,
Loading
Loading
@@ -49,9 +47,5 @@ module StatusPage
storage_client: storage_client
).execute
end
def attachements_enabled?
Feature.enabled?(:status_page_attachments, @project, default_enabled: true)
end
end
end
---
name: status_page_attachments
introduced_by_url:
rollout_issue_url:
group:
type: development
default_enabled: true
Loading
Loading
@@ -39,17 +39,6 @@ RSpec.describe StatusPage::PublishDetailsService do
allow(storage_client).to receive(:list_object_keys).and_return([])
end
 
context 'when feature flag disabled' do
before do
stub_feature_flags(status_page_attachments: false)
end
it 'does not publish attachments and returns success' do
expect(StatusPage::PublishAttachmentsService).not_to receive(:new)
expect(subject.success?).to be true
end
end
context 'when successful' do
let(:success_response) { double(error?: false, success?: true) }
 
Loading
Loading
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment