Skip to content

Support for large workflow results#127

Merged
dmugtasimov merged 1 commit intomasterfrom
dmugtasimov/aed-154-update-js-sdk
Apr 28, 2025
Merged

Support for large workflow results#127
dmugtasimov merged 1 commit intomasterfrom
dmugtasimov/aed-154-update-js-sdk

Conversation

@dmugtasimov
Copy link
Copy Markdown
Contributor

No description provided.

@dmugtasimov dmugtasimov marked this pull request as ready for review April 21, 2025 16:02
@dmugtasimov dmugtasimov requested a review from jzone3 as a code owner April 21, 2025 16:02
@dmugtasimov dmugtasimov force-pushed the dmugtasimov/aed-154-update-js-sdk branch from 245de35 to 08f4ac2 Compare April 21, 2025 16:14
Comment thread src/utils.ts
}, timeout);

promise.then((result) => {
clearTimeout(timer);
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You can clear the timeout in finally block

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No, I can't because timer is not available in finally {} scope, and setTimeout() cannot be moved to outer scope, because it requires reject

@dmugtasimov dmugtasimov force-pushed the dmugtasimov/aed-154-update-js-sdk branch from 08f4ac2 to 41f605f Compare April 28, 2025 12:49
@dmugtasimov dmugtasimov force-pushed the dmugtasimov/aed-154-update-js-sdk branch from 41f605f to 51649f4 Compare April 28, 2025 13:21
@dmugtasimov dmugtasimov merged commit 42ffddd into master Apr 28, 2025
6 checks passed
@dmugtasimov dmugtasimov deleted the dmugtasimov/aed-154-update-js-sdk branch April 28, 2025 13:27
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