Commit d42291e9 authored by Aakriti Gupta's avatar Aakriti Gupta Committed by Bob Van Landuyt

Expose new members count in Group Activity API

parent fcf2ee20
......@@ -120,7 +120,7 @@ The following API resources are available outside of project and group contexts
| [Events](events.md) | `/events`, `/users/:id/events` (also available for projects) |
| [Feature flags](features.md) | `/features` |
| [Geo Nodes](geo_nodes.md) **(PREMIUM ONLY)** | `/geo_nodes` |
| [Group Activity Analytics](group_activity_analytics.md) **(STARTER)** | `/analytics/group_activity/{issues_count | merge_requests_count}` |
| [Group Activity Analytics](group_activity_analytics.md) **(STARTER)** | `/analytics/group_activity/{issues_count | merge_requests_count | new_members_count }` |
| [Import repository from GitHub](import.md) | `/import/github` |
| [Issues](issues.md) | `/issues` (also available for groups and projects) |
| [Issues Statistics](issues_statistics.md) | `/issues_statistics` (also available for groups and projects) |
......
......@@ -49,3 +49,27 @@ Example response:
```json
{ "merge_requests_count": 10 }
```
## Get count of members recently added to group
```plaintext
GET /analytics/group_activity/new_members_count
```
Parameters:
| Attribute | Type | Required | Description |
| --------- | ---- | -------- | ----------- |
| `group_path` | string | yes | Group path |
Example request:
```shell
curl --header "PRIVATE-TOKEN: <your_access_token>" https://gitlab.example.com/api/v4/analytics/group_activity/new_members_count?group_path=gitlab-org
```
Example response:
```json
{ "new_members_count": 10 }
```
---
title: Add API endpoint for new members' count in Group Activity Analytics
merge_request: 26601
author:
type: added
......@@ -9,7 +9,6 @@ module API
before do
authenticate!
not_found! unless Feature.enabled?(:group_activity_analytics)
end
helpers do
......@@ -35,7 +34,11 @@ module API
end
get 'issues_count' do
not_found! unless
Feature.enabled?(:group_activity_analytics, group)
authorize! :read_group_activity_analytics, group
present(
calculator,
with: EE::API::Entities::Analytics::GroupActivity::IssuesCount
......@@ -52,12 +55,37 @@ module API
end
get 'merge_requests_count' do
not_found! unless
Feature.enabled?(:group_activity_analytics, group)
authorize! :read_group_activity_analytics, group
present(
calculator,
with: EE::API::Entities::Analytics::GroupActivity::MergeRequestsCount
)
end
desc 'Get count of recently created group members' do
detail DESCRIPTION_DETAIL
success EE::API::Entities::Analytics::GroupActivity::NewMembersCount
end
params do
requires :group_path, type: String, desc: 'Group Path'
end
get 'new_members_count' do
not_found! unless
Feature.enabled?(:group_activity_analytics, group)
authorize! :read_group_activity_analytics, group
present(
calculator,
with: EE::API::Entities::Analytics::GroupActivity::NewMembersCount
)
end
end
end
end
......
# frozen_string_literal: true
module EE
module API
module Entities
module Analytics
module GroupActivity
class NewMembersCount < Grape::Entity
expose :new_members_count
end
end
end
end
end
end
......@@ -16,10 +16,20 @@ describe EE::API::Entities::Analytics::GroupActivity do
end
context 'issues count' do
it_behaves_like 'exposes count', 'issues_count', EE::API::Entities::Analytics::GroupActivity::IssuesCount
it_behaves_like 'exposes count',
'issues_count',
EE::API::Entities::Analytics::GroupActivity::IssuesCount
end
context 'merge requests count' do
it_behaves_like 'exposes count', 'merge_requests_count', EE::API::Entities::Analytics::GroupActivity::MergeRequestsCount
it_behaves_like 'exposes count',
'merge_requests_count',
EE::API::Entities::Analytics::GroupActivity::MergeRequestsCount
end
context 'new members count' do
it_behaves_like 'exposes count',
'new_members_count',
EE::API::Entities::Analytics::GroupActivity::NewMembersCount
end
end
......@@ -11,7 +11,7 @@ describe API::Analytics::GroupActivityAnalytics do
let_it_be(:anonymous_user) { create(:user) }
shared_examples 'GET group_activity' do |activity|
shared_examples 'GET group_activity' do |activity, count|
let(:feature_enabled) { true }
let(:params) { { group_path: group.full_path } }
let(:current_user) { reporter }
......@@ -29,7 +29,7 @@ describe API::Analytics::GroupActivityAnalytics do
end
it 'is returns a count' do
expect(response.parsed_body).to eq({ "#{activity}_count" => 0 })
expect(response.parsed_body).to eq({ "#{activity}_count" => count })
end
context 'when feature is not available in plan' do
......@@ -58,10 +58,14 @@ describe API::Analytics::GroupActivityAnalytics do
end
context 'GET /group_activity/issues_count' do
it_behaves_like 'GET group_activity', 'issues'
it_behaves_like 'GET group_activity', 'issues', 0
end
context 'GET /group_activity/merge_requests_count' do
it_behaves_like 'GET group_activity', 'merge_requests'
it_behaves_like 'GET group_activity', 'merge_requests', 0
end
context 'GET /group_activity/new_members_count' do
it_behaves_like 'GET group_activity', 'new_members', 1 # reporter
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