Skip to content

Commit c85e784

Browse files
committed
fix: include orgUid in rejected-import warning for default organizations
Promise.allSettled results share indices with the pending array; correlate them so the warning log identifies exactly which org failed.
1 parent f625c95 commit c85e784

2 files changed

Lines changed: 6 additions & 6 deletions

File tree

src/tasks/sync-default-organizations-v2.js

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -84,13 +84,13 @@ const task = new Task('sync-default-organizations-v2', async () => {
8484
}),
8585
);
8686

87-
for (const result of results) {
87+
results.forEach((result, i) => {
8888
if (result.status === 'rejected') {
8989
loggerV2.warn(
90-
`[v2]: Failed to import a default organization: ${result.reason?.message || result.reason}. Will retry on next task run.`,
90+
`[v2]: Failed to import default organization ${pending[i]}: ${result.reason?.message || result.reason}. Will retry on next task run.`,
9191
);
9292
}
93-
}
93+
});
9494
}
9595
} catch (error) {
9696
loggerV2.error(

src/tasks/sync-default-organizations.js

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -73,13 +73,13 @@ const task = new Task('sync-default-organizations', async () => {
7373
}),
7474
);
7575

76-
for (const result of results) {
76+
results.forEach((result, i) => {
7777
if (result.status === 'rejected') {
7878
logger.warn(
79-
`[v1]: Failed to import a default organization: ${result.reason?.message || result.reason}. Will retry on next task run.`,
79+
`[v1]: Failed to import default organization ${pending[i]}: ${result.reason?.message || result.reason}. Will retry on next task run.`,
8080
);
8181
}
82-
}
82+
});
8383
}
8484
} catch (error) {
8585
logger.error(

0 commit comments

Comments
 (0)