Skip to content

Add investigation reason to action-account output (LG-12212)#9979

Merged
zachmargolis merged 2 commits intomainfrom
margolis-action-account-reason
Jan 26, 2024
Merged

Add investigation reason to action-account output (LG-12212)#9979
zachmargolis merged 2 commits intomainfrom
margolis-action-account-reason

Conversation

@zachmargolis
Copy link
Contributor

We got a request to include the reason (which is currently logged to Slack) in the output of the command line as well

Before

+--------------------------------------+--------------------------------------------------+
| uuid                                 | status                                           |
+--------------------------------------+--------------------------------------------------+
| 770059eb-eacb-4738-9e0d-e39091c3372b | Error: User does not have a pending fraud review |
+--------------------------------------+--------------------------------------------------+

After

+--------------------------------------+--------------------------------------------------+---------+
| uuid                                 | status                                           | reason  |
+--------------------------------------+--------------------------------------------------+---------+
| 770059eb-eacb-4738-9e0d-e39091c3372b | Error: User does not have a pending fraud review | INV1234 |
+--------------------------------------+--------------------------------------------------+---------+

changelog: Internal, Logging, Add reason to admin action output
@zachmargolis zachmargolis requested review from a team January 26, 2024 00:28
Copy link
Contributor

@matthinz matthinz left a comment

Choose a reason for hiding this comment

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

LGTM

@zachmargolis zachmargolis merged commit d36e602 into main Jan 26, 2024
@zachmargolis zachmargolis deleted the margolis-action-account-reason branch January 26, 2024 17:20
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