Skip to content

fix(sdk): centicredits in executions list#63

Merged
Andrii Chumak (andriichumak) merged 2 commits into
mainfrom
ach/WM-4172
Jun 28, 2026
Merged

fix(sdk): centicredits in executions list#63
Andrii Chumak (andriichumak) merged 2 commits into
mainfrom
ach/WM-4172

Conversation

@andriichumak

Copy link
Copy Markdown
Contributor

Document centicredits field in the types and test assets. The field is already being delivered, was not documented.

https://make.atlassian.net/browse/WM-4172

Document centicredits field in the types and test assets. The field is already being delivered, was not documented.

https://make.atlassian.net/browse/WM-4172
Copilot AI review requested due to automatic review settings June 28, 2026 15:38
@andriichumak Andrii Chumak (andriichumak) requested a review from a team as a code owner June 28, 2026 15:38

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Documents the already-delivered centicredits field for scenario executions by updating the SDK’s Execution type and aligning execution mock responses used by unit tests.

Changes:

  • Added centicredits to the Execution TypeScript type (with JSDoc).
  • Updated execution list/get JSON mocks to include centicredits.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

File Description
src/endpoints/executions.ts Documents centicredits on the Execution type.
test/mocks/executions/list.json Adds centicredits to the mocked list response payload.
test/mocks/executions/get.json Adds centicredits to the mocked get response payload.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread src/endpoints/executions.ts
@andriichumak Andrii Chumak (andriichumak) merged commit 21817cb into main Jun 28, 2026
4 checks passed
@andriichumak Andrii Chumak (andriichumak) deleted the ach/WM-4172 branch June 28, 2026 16:54
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.

4 participants