Skip to content

Fixes namespace_id for user with no groups import

Catalin Irimie requested to merge cat-fix-namespaceid-import-39078 into master

What does this MR do?

User imports fail when the user is not part of a group, related to !20405 (merged) when we've removed a duplicate hidden_field_tag though I've missed that the original (remaining) hidden_field_tag was incorrect, this fixes it.

Related to #39078 (closed).

Screenshots

Because the current hidden_field_tag passes the value as value: id, it is reflected in the UI like this, invalid:

image

After this MR, it changes to what should be expected:

image

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 🤖 GitLab Bot 🤖

Merge request reports