Commit 2dbad45e authored by Justin Ho's avatar Justin Ho
parent 25b79037
...@@ -52,14 +52,14 @@ module ServicesHelper ...@@ -52,14 +52,14 @@ module ServicesHelper
end end
def service_save_button(service) def service_save_button(service)
button_tag(class: 'btn btn-success', type: 'submit', disabled: service.deprecated?, data: { qa_selector: 'save_changes_button' }) do button_tag(class: 'btn btn-success', type: 'submit', data: { qa_selector: 'save_changes_button' }) do
icon('spinner spin', class: 'hidden js-btn-spinner') + icon('spinner spin', class: 'hidden js-btn-spinner') +
content_tag(:span, 'Save changes', class: 'js-btn-label') content_tag(:span, 'Save changes', class: 'js-btn-label')
end end
end end
def disable_fields_service?(service) def disable_fields_service?(service)
!current_controller?("admin/services") && service.deprecated? !current_controller?('admin/services')
end end
def scoped_integrations_path def scoped_integrations_path
......
...@@ -345,10 +345,6 @@ class Service < ApplicationRecord ...@@ -345,10 +345,6 @@ class Service < ApplicationRecord
service service
end end
def deprecated?
false
end
# override if needed # override if needed
def supports_data_fields? def supports_data_fields?
false false
......
...@@ -2768,8 +2768,6 @@ Service ...@@ -2768,8 +2768,6 @@ Service
updates the has_external_issue_tracker boolean updates the has_external_issue_tracker boolean
on update on update
updates the has_external_issue_tracker boolean updates the has_external_issue_tracker boolean
#deprecated?
should return false by default
#api_field_names #api_field_names
filters out sensitive fields filters out sensitive fields
......
...@@ -523,15 +523,6 @@ describe Service do ...@@ -523,15 +523,6 @@ describe Service do
end end
end end
describe "#deprecated?" do
let(:project) { create(:project, :repository) }
it 'returns false by default' do
service = create(:service, project: project)
expect(service.deprecated?).to be_falsy
end
end
describe '#api_field_names' do describe '#api_field_names' do
let(:fake_service) do let(:fake_service) do
Class.new(Service) do Class.new(Service) do
......
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