FIX: Mobile group add dropdown was broken

It looks like we removed the bulk add modal but didn't update the mobile template.
discourse · Aug 15, 2020 · 58ee947 · 58ee947
1 parent 51a6e96
commit 58ee947
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 11 deletions.
diff --git a/app/assets/javascripts/discourse/app/templates/mobile/group-index.hbs b/app/assets/javascripts/discourse/app/templates/mobile/group-index.hbs
@@ -10,8 +10,8 @@
     {{#if canManageGroup}}
       {{#if currentUser.admin}}
         {{group-members-dropdown
+          groupName=model.full_name
           showAddMembersModal=(route-action "showAddMembersModal")
-          showBulkAddModal=(route-action "showBulkAddModal")
         }}
       {{else}}
         {{d-button

diff --git a/app/assets/javascripts/select-kit/addon/components/group-members-dropdown.js b/app/assets/javascripts/select-kit/addon/components/group-members-dropdown.js
@@ -14,19 +14,12 @@ export default DropdownSelectBoxComponent.extend({
     const items = [
       {
         id: "showAddMembersModal",
-        name: I18n.t("groups.add_members.title"),
+        name: I18n.t("groups.add_members.title", {
+          group_name: this.groupName
+        }),
         icon: "user-plus"
       }
     ];
-
-    if (this.currentUser.admin) {
-      items.push({
-        id: "showBulkAddModal",
-        name: I18n.t("admin.groups.bulk_add.title"),
-        icon: "users"
-      });
-    }
-
     return items;
   }),