Commit 137aaf1c authored by Mayra Cabrera's avatar Mayra Cabrera

Merge branch 'remove_api_activity_logging_feature_flag' into 'master'

Remove api_activity_logging feature flag

See merge request gitlab-org/gitlab!28755
parents 73d22fa2 7634deb0
---
title: Enable last user activity logging on the REST API
merge_request: 28755
author:
type: added
......@@ -105,7 +105,7 @@ module API
namespace do
after do
::Users::ActivityService.new(@current_user).execute if Feature.enabled?(:api_activity_logging)
::Users::ActivityService.new(@current_user).execute
end
# Keep in alphabetical order
......
......@@ -13,13 +13,5 @@ describe API::API do
it 'updates the users last_activity_on date' do
expect { get api('/groups', user) }.to change { user.reload.last_activity_on }.to(Date.today)
end
context 'when the the api_activity_logging feature is disabled' do
it 'does not touch last_activity_on' do
stub_feature_flags(api_activity_logging: false)
expect { get api('/groups', user) }.not_to change { user.reload.last_activity_on }
end
end
end
end
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