Skip to content

Update Add Members Api to accept user_id list

Jackie Fraser requested to merge update-members-api-for-multiple-user_ids into master

What does this MR do?

Updates Add Member API post :id/members to accept a list of IDs for user_id. Previously only accepted one.

Related to !42398 (merged)

Related to https://gitlab.com/gitlab-org/gitlab/-/issues/217922

Screenshots

Using user_id for multiple IDs Response:
user_id_commas toast-success
user_id for single id (old behavior) Responses:
Screen_Shot_2020-10-01_at_3.58.04_PM Success: user_id Error: error-user_id

Does this MR meet the acceptance criteria?

Conformity

Availability and Testing

Security

If this MR contains changes to processing or storing of credentials or tokens, authorization and authentication methods and other items described in the security review guidelines:

  • Label as security and @ mention @gitlab-com/gl-security/appsec
  • The MR includes necessary changes to maintain consistency between UI, API, email, or other methods
  • Security reports checked/validated by a reviewer from the AppSec team
Edited by Jackie Fraser

Merge request reports

Loading