Skip to content

Add basic PlaceholdersTabApp and feature flag

Justin Ho Tuan Duong requested to merge 454964-user-mapping-fe-add-basic-app-structure-and-components-ff into master

What does this MR do and why?

To be used for placeholder users from import by direct transfer. We also add the bulk_import_user_mapping feature flag which will be used for the whole feature.

The tab is not shown at the moment as showTab returns false for the placeholder tab. It will only show when we send data from the backend helpers.

MR acceptance checklist

Please evaluate this MR against the MR acceptance checklist. It helps you analyze changes to reduce risks in quality, performance, reliability, security, and maintainability.

Screenshots or screen recordings

User_mapping_basic_app

How to set up and validate locally

  1. Go to Group > Manage > Members. For example, http://127.0.0.1:3000/groups/gitlab-org/-/group_members.
  2. Apply the following diff:
diff --git a/app/assets/javascripts/members/components/members_tabs.vue b/app/assets/javascripts/members/components/members_tabs.vue
index 431f027936c7..fb0fc6c47fb6 100644
--- a/app/assets/javascripts/members/components/members_tabs.vue
+++ b/app/assets/javascripts/members/components/members_tabs.vue
@@ -26,6 +26,7 @@ export default {
           ...getters,
           // eslint-disable-next-line @gitlab/require-i18n-strings
           [`${memberType}Count`](state) {
+            if (memberType === 'placeholder') return 1;
             return countComputed(state, memberType);
           },
         };

Related to #454964 (closed)

Merge request reports