Skip to content
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.

Sign up for GitHub

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

Jump to bottom

[Feature Request] SDK clients should set a UpdateID on any update request even if the user did not specify one #198

Closed
Quinn-With-Two-Ns opened this issue Mar 6, 2024 · 1 comment · Fixed by #251
Closed

[Feature Request] SDK clients should set a UpdateID on any update request even if the user did not specify one #198

Quinn-With-Two-Ns opened this issue Mar 6, 2024 · 1 comment · Fixed by #251
Labels
enhancement New feature or request

Comments

@Quinn-With-Two-Ns
Copy link

See: temporalio/features#431
@Quinn-With-Two-Ns Quinn-With-Two-Ns added the enhancement New feature or request label Mar 6, 2024
@Quinn-With-Two-Ns Quinn-With-Two-Ns mentioned this issue Mar 6, 2024
Closed
6 tasks
@cretz
Copy link
Member

cretz commented Mar 18, 2024

Also make sure that update ID can be set in update options constructor

cretz added a commit to cretz/temporal-sdk-dotnet that referenced this issue May 21, 2024
@cretz
Required wait update stage and polling improvements
7c56079 
Fixes temporalio#198
Fixes temporalio#199
Fixes temporalio#231
@cretz cretz mentioned this issue May 21, 2024
Merged
cretz added a commit to cretz/temporal-sdk-dotnet that referenced this issue May 21, 2024
@cretz
Required wait update stage and polling improvements
4849264 
Fixes temporalio#198
Fixes temporalio#199
Fixes temporalio#231
cretz added a commit that referenced this issue May 21, 2024
@cretz
Required wait update stage and polling improvements (#251)
618e878 
Fixes #198
Fixes #199
Fixes #231
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Required wait update stage and polling improvements cretz/temporal-sdk-dotnet
2 participants
@cretz @Quinn-With-Two-Ns