Skip to content

Conversation

@CahidArda
Copy link
Collaborator

@CahidArda CahidArda commented Feb 26, 2025

Adds workflow parameter to context.call which will call the workflow in a typesafe way

const result = await context.call("call other workflow", {
  workflow: workflowOne,
  body: 2,
});

@linear
Copy link

linear bot commented Feb 26, 2025

@CahidArda CahidArda marked this pull request as ready for review March 7, 2025 10:13
test was checking the result of context.notify in a loop, which resulted in different number of calls. fixed it by waiting for a long while and notifying once.
@CahidArda CahidArda changed the title DX-1691: Add waitForResult parameter to invoke DX-1691: Add workflow parameter to context.call Mar 12, 2025
@yunusemreozdemir
Copy link
Contributor

Can you provide some information on why the tests are failing?

@CahidArda
Copy link
Collaborator Author

we are getting errors because of timeouts caused by QStash. Trying to pass them with retries.

@CahidArda CahidArda merged commit cb28b8e into main Mar 21, 2025
17 of 20 checks passed
@CahidArda CahidArda deleted the DX-1691-waitForResult-in-invoke branch March 21, 2025 12:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants