Skip to content
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.

Sign up for GitHub

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

Jump to bottom

fix(api): delete useless unmarshal for outgoing payload #4866

Merged
merged 4 commits into from
Jan 27, 2020
Merged

fix(api): delete useless unmarshal for outgoing payload #4866

merged 4 commits into from
Jan 27, 2020

Conversation

bnjjj
Copy link
Contributor

@bnjjj bnjjj commented Jan 6, 2020

Signed-off-by: Benjamin Coenen 5719034+bnjjj@users.noreply.github.com

  1. Description
  2. Related issues
  3. About tests
  4. Mentions
    cc @blaryjp

@ovh/cds

@bnjjj
fix(api): delete useless unmarshal for outgoing payload
710e3d0 
Signed-off-by: Benjamin Coenen <5719034+bnjjj@users.noreply.github.com>
@ovh-cds
Copy link
Collaborator

ovh-cds commented Jan 6, 2020

CDS Report build-all-cds#11934.0 ✘

  • Build
    • Build and Package All ✔
    • Build Debpacker ✔
    • Build OS-Ansible-Inventory ✔
    • Build SMTPMock ✘
    • Lint UI Translate ✔
    • Test CLI ✔
    • Test Engine ✔
    • Test SDK ✔
    • Test UI ✔

@ovh-cds
Copy link
Collaborator

ovh-cds commented Jan 27, 2020

CDS Report build-all-cds#12190.0 ✘

  • Build
    • Build and Package All ✘
    • Build Debpacker ✔
    • Build OS-Ansible-Inventory ✔
    • Build SMTPMock ✔
    • Lint UI Translate ✔
    • Test CLI ✔
    • Test Engine ✘
    • Test SDK ✔
    • Test UI ✔

@bnjjj
test(sdk): generate new mock sdk
d46ebea 
Signed-off-by: Benjamin Coenen <5719034+bnjjj@users.noreply.github.com>
@bnjjj bnjjj merged commit 0e448fd into master Jan 27, 2020
@bnjjj bnjjj deleted the fix_outgoing_payload branch January 31, 2020 14:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@yesnault yesnault yesnault approved these changes

@fsamin fsamin fsamin approved these changes

@sguiheux sguiheux sguiheux approved these changes

@richardlt richardlt richardlt approved these changes

Assignees
No one assigned
Labels
api bug ready-to-merge
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@bnjjj @ovh-cds @yesnault @fsamin @sguiheux @richardlt