Skip to content

Add status changed by user info to context #5459

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 3 commits into from
Dec 2, 2021

Conversation

khushboobhatia01
Copy link
Contributor

This is required to track who cancelled/paused/resumed any execution.

"context" : { "user" : "stanley", "pack" : "examples", "workflow_execution" : "61a0887aac2049b57fed57de", "paused_by" : "stanley", "resumed_by" : "stanley" }

@pull-request-size pull-request-size bot added the size/S PR that changes 10-29 lines. Very easy to review. label Nov 26, 2021
@arm4b arm4b requested review from Kami, m4dcoder and a team November 26, 2021 19:23
Copy link
Member

@arm4b arm4b left a comment

Choose a reason for hiding this comment

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

A really helpful addition 👍

We'll need a small Changelog record for this anyway and I'm sure someone else will request unit tests as well. So adding those would be nice.

@pull-request-size pull-request-size bot added size/M PR that changes 30-99 lines. Good size to review. and removed size/S PR that changes 10-29 lines. Very easy to review. labels Nov 27, 2021
@khushboobhatia01
Copy link
Contributor Author

A really helpful addition 👍

We'll need a small Changelog record for this anyway and I'm sure someone else will request unit tests as well. So adding those would be nice.

Addressed @armab

@arm4b arm4b added this to the 3.7.0 milestone Nov 27, 2021
@arm4b arm4b merged commit 6bcecbb into StackStorm:master Dec 2, 2021
arm4b added a commit that referenced this pull request Dec 9, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
size/M PR that changes 30-99 lines. Good size to review.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants