Ryan/documenso
1
0
Fork 1
mirror of https://github.com/documenso/documenso.git synced 2025-11-10 20:42:34 +10:00
Code Issues Packages Projects Releases Wiki Activity

fix: resolve issue with sealing task on inngest (#2146)

Browse Source 
Currently on inngest the sealing task fails during decoration stating
that it can not find the step "xxx"

My running theory is that this was due to it being a
Promise.all(map(...)) even though that isn't explicitly disallowed.

This change turns it into a for loop collecting promises to be awaited
after the fact.

Local inngest testing looks promising.
This commit is contained in:
Lucas Smith
2025-11-08 00:48:13 +11:00
committed by GitHub
parent 3d539b20ad
commit db5524f8ce
1 changed files with 26 additions and 21 deletions
Download Patch File Download Diff File Expand all files Collapse all files

17
packages/lib/jobs/definitions/internal/seal-document.handler.ts
View File

@ -189,9 +189,11 @@ export const run = async ({
settings,
});
const newDocumentData = await Promise.all(
envelopeItems.map(async (envelopeItem) =>
io.runTask(`decorate-${envelopeItem.id}`, async () => {
const decoratePromises: Array<Promise<{ oldDocumentDataId: string; newDocumentDataId: string }>> =
[];
for (const envelopeItem of envelopeItems) {
const task = io.runTask(`decorate-${envelopeItem.id}`, async () => {
const envelopeItemFields = envelope.envelopeItems.find(
(item) => item.id === envelopeItem.id,
)?.field;
@ -209,9 +211,12 @@ export const run = async ({
certificateData,
auditLogData,
});
}),
),
);
});
decoratePromises.push(task);
}
const newDocumentData = await Promise.all(decoratePromises);
const postHog = PostHogServerClient();