Skip to content

Fix TypeError when resuming process through the callback endpoint #948

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
May 20, 2025

Conversation

Haak1981
Copy link
Contributor

Getting a TypeError: _celery_resume_process() missing 2 required keyword-only arguments: 'user_inputs' and 'user'. Scenario is when using callback in combination with Celery. When using threadpool execution context the issue is not seen as there a default (None) is set.

image

Getting a TypeError: _celery_resume_process() missing 2 required keyword-only arguments: 'user_inputs' and 'user'. Scenario is when using callback in combination with Celery. When using threadpool execution context the issue is not seen as there a default (None) is set.
Copy link

codspeed-hq bot commented May 19, 2025

CodSpeed Performance Report

Merging #948 will not alter performance

Comparing Haak1981:patch-1 (a47f1ad) with main (ebbeb34)

Summary

✅ 12 untouched benchmarks

@pboers1988 pboers1988 requested a review from Mark90 May 19, 2025 19:29
Copy link
Member

@pboers1988 pboers1988 left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@Mark90 Mark90 left a comment

Choose a reason for hiding this comment

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

Looks correct to me, continue_awaiting_process indeed doesn't have these variables to pass along.

Thanks for taking the time to dig into this :)

@Mark90 Mark90 changed the title Update celery.py Fix TypeError when resuming process through the callback endpoint May 19, 2025
@pboers1988 pboers1988 merged commit 298c4f3 into workfloworchestrator:main May 20, 2025
14 checks passed
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.

3 participants