@crowdin/app-project-module 0.71.1 → 0.71.2

package/out/modules/integration/util/defaults.js

@@ -147,7 +147,7 @@ function applyIntegrationModuleDefaults(config, integration) {
             fields = yield getUserSettings(projectId, crowdinClient, integrationCredentials);
         }
         const defaultSettings = [];
-        const mangers = yield getManagers(projectId, crowdinClient, integrationCredentials);
+        const mangers = yield getManagers(crowdinClient, projectId, integrationCredentials.ownerId);
         if (mangers.length) {
             defaultSettings.push({
                 key: 'managers',
@@ -320,17 +320,30 @@ function getOAuthLoginFormId(clientId) {
     return `oauth_form_${clientId}`;
 }
 exports.getOAuthLoginFormId = getOAuthLoginFormId;
-function getManagers(projectId, client, integrationCredentials) {
+function getManagers(client, projectId, ownerId) {
     return __awaiter(this, void 0, void 0, function* () {
         const managers = [];
         if (client.organization) {
-            const admins = (yield client.usersApi.listUsers()).data.filter((user) => user.data.isAdmin);
-            managers.push(...admins.map((admin) => admin.data));
+            try {
+                const admins = (yield client.usersApi.withFetchAll().listUsers()).data.filter((user) => user.data.isAdmin);
+                managers.push(...admins.map((admin) => admin.data));
+            }
+            catch (e) {
+                console.error('Failed to get organization users', e);
+            }
         }
-        const projectMembers = (yield client.usersApi.listProjectMembers(projectId)).data.filter((user) => 'role' in user.data ? ['owner', 'manager'].includes(user.data.role) : user.data.isManager);
+        const projectMembers = (yield client.usersApi.withFetchAll().listProjectMembers(projectId)).data.filter((user) => 'role' in user.data ? ['owner', 'manager'].includes(user.data.role) : user.data.isManager);
         managers.push(...projectMembers.map((members) => members.data));
-        return managers
-            .filter((manager) => manager.id !== +integrationCredentials.ownerId)
-            .filter((manager, index, self) => self.findIndex((m) => m.id === manager.id) === index);
+        const uniqueManagers = new Map();
+        managers.forEach((manager) => {
+            if (manager.id !== ownerId && !uniqueManagers.has(manager.id)) {
+                uniqueManagers.set(manager.id, manager);
+            }
+        });
+        return Array.from(uniqueManagers.values()).sort((a, b) => {
+            const aValue = a.firstName || a.username || '';
+            const bValue = b.firstName || b.username || '';
+            return aValue.localeCompare(bValue);
+        });
     });
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@crowdin/app-project-module",
-  "version": "0.71.1",
+  "version": "0.71.2",
   "description": "Module that generates for you all common endpoints for serving standalone Crowdin App",
   "main": "out/index.js",
   "types": "out/index.d.ts",