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.

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

Valid states for runtimeStatus #3989

Merged
merged 2 commits into from
Feb 1, 2024
Merged

Valid states for runtimeStatus #3989

merged 2 commits into from
Feb 1, 2024

Conversation

hhunter-ms
Copy link
Collaborator

Description

Added the valid states for workflow runtimeStatus

Issue reference

PR will close: #3904

@hhunter-ms hhunter-ms added this to the 1.12 milestone Jan 31, 2024
@hhunter-ms hhunter-ms self-assigned this Jan 31, 2024
@hhunter-ms hhunter-ms requested review from a team as code owners January 31, 2024 16:59
@@ -235,7 +235,7 @@ The API call will provide a JSON response similar to this:

Parameter | Description
--------- | -----------
`runtimeStatus` | The status of the workflow instance. Values include: `RUNNING`, `TERMINATED`, `PAUSED`
`runtimeStatus` | The status of the workflow instance. Values include: `"RUNNING"`, `"COMPLETED"`, `"CONTINUED_AS_NEW"`, `"FAILED"`, `"CANCELED"`, `"TERMINATED"`, `"PENDING"`, `"SUSPENDED"`
Copy link
Member

@msfussell msfussell Jan 31, 2024

Choose a reason for hiding this comment

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

I presume this is "CANCELLED" and not "CANCELED"?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

TIL from Grammarly:

Canceled or cancelled is the past tense of the verb to cancel. Both spellings are correct; Americans favor canceled (one l), while cancelled (two l's) is preferred in British English and other dialects. Canceling/cancelling and canceler/canceller follow the same pattern.

So either way works for me haha

Copy link
Contributor

Choose a reason for hiding this comment

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

Copy link
Member

@msfussell msfussell left a comment

Choose a reason for hiding this comment

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

LGTM

@msfussell msfussell merged commit e4847eb into dapr:v1.12 Feb 1, 2024
7 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.

Valid workflow states needs update
3 participants