-
Notifications
You must be signed in to change notification settings - Fork 429
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add the ability to assign a feature to a user group #2666
Comments
The third criteria was moved into a new ticket by @dabeeeenster the remaining functionality is here: |
The API endpoints are in the following format. To list the groups in an organisation to retrieve the group
[
{
"id": 1,
"name": "TestGroup",
},
{
"id": 2,
"name": "TestGroup2",
}
] Adding a group to an assigned feature:
{"group_ids": [1, 2]} Removing a group from being assigned to a feature:
{"group_ids": [1, 2]} For both endpoints, the response body looks the same, which represents all the now-existing group owners for the feature. {
"group_owners": [
{
"id": 1,
"name": "GroupName1"
},
{
"id": 2,
"name": "GroupName2"
}
]
} To pre-load the group owners for a list of features, find the existing URL structure for the following pattern, with an abbreviated response body for brevity:
{
"count": 2,
"next": null,
"previous": null,
"results": [
{
"id": 77,
"name": "test_feature_empty",
"group_owners": [],
},
{
"id": 78,
"name": "test_feature",
"group_owners": [
{
"id": 6,
"name": "Test Group"
},
{
"id": 7,
"name": "Second Group"
}
],
}
]
} |
@kyle-ssg just noticed that (unless I'm missing something) this isn't using the new endpoint to retrieve the groups, which will mean that only organisation admins can use this feature.
Please use the above endpoint for this purpose. All users of an organisation will have access to this data. |
Functionality should be as per the current functionality to assign a feature to a user, but for groups. A feature should be able to be assigned to multiple users and multiple groups.
Acceptance Criteria
The text was updated successfully, but these errors were encountered: