Skip to content

updated TaskSatus enum#1432

Merged
andrewelamb merged 1 commit into
developfrom
add_task_stauses
Jul 24, 2026
Merged

updated TaskSatus enum#1432
andrewelamb merged 1 commit into
developfrom
add_task_stauses

Conversation

@andrewelamb

@andrewelamb andrewelamb commented Jul 24, 2026

Copy link
Copy Markdown
Contributor

Problem:

  • The TaskState enum in synapseclient/models/curation.py was out of sync with the Synapse REST API.
  • The API documents six task states, but the client only defined four (NOT_STARTED, IN_PROGRESS, COMPLETED, CANCELED).
  • The two missing states — EXECUTING and IN_REVIEW — are returned by the API when a task has an automated async job running or awaiting review. Deserializing a CurationTask in one of these states would fail to coerce into a valid TaskState.
  • Reference: TaskState

Solution:

  • Added the two missing enum members to TaskState, matching the names, values, and ordering in the REST docs:
    • EXECUTING — "An automated execution (async job) is currently running for this task."
    • IN_REVIEW — "The automated execution completed successfully and the results are pending human review."
  • Each member carries the documented description as its docstring, consistent with the existing states.

@andrewelamb
andrewelamb requested a review from a team as a code owner July 24, 2026 15:48
@andrewelamb
andrewelamb merged commit 41ea6b9 into develop Jul 24, 2026
17 of 23 checks passed
@andrewelamb
andrewelamb deleted the add_task_stauses branch July 24, 2026 16:17
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