Skip to content

Conversation

@gow
Copy link
Contributor

@gow gow commented Jan 6, 2026

What was changed

temporal workflow describe now shows pause information if the workflow is paused.

Why?

When a workflow is paused we need to show the relevant pause information like paused time, identity & reason.

Checklist

  1. Closes

  2. How was this tested:
    Tested manually

  3. Any docs updates needed?
    N/A

@gow gow requested review from a team as code owners January 6, 2026 06:14
@gow gow changed the title Cg/wf pause 1 Show pause information in "workflow describe" command Jan 6, 2026
@gow gow requested a review from spkane31 January 6, 2026 06:15
WorkflowId: c.WorkflowId,
RunId: c.RunId,
Identity: c.Parent.Identity,
Reason: c.Reason,
Copy link
Contributor

Choose a reason for hiding this comment

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

could revert but not required

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Sorry I forgot to add a note. This is needed to fix a bug. I was not capturing the reason in pause command.

Copy link
Contributor

Choose a reason for hiding this comment

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

Why's that? This looks like it just moves the reason to a different line

Copy link
Contributor Author

Choose a reason for hiding this comment

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

oh! yeah. That explains it. I might have accidentally removed this line locally and ran into the bug. And later I added it thinking I fixed it 🤦

Copy link
Contributor

Choose a reason for hiding this comment

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

not worth making the change, just pointed it out. no biggie

@gow gow merged commit 5e6f828 into next-server Jan 6, 2026
6 of 7 checks passed
@gow gow deleted the cg/wf_pause_1 branch January 6, 2026 17:34
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