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

Move the "transformHeaders" into a shared http library -1 #848

Closed
4 of 13 tasks
elnyry-sam-k opened this issue Jul 5, 2019 · 0 comments
Closed
4 of 13 tasks

Move the "transformHeaders" into a shared http library -1 #848

elnyry-sam-k opened this issue Jul 5, 2019 · 0 comments
Assignees
@mdebarros @rmothilal
Labels
story
Milestone
Sprint 7.1

Comments

@elnyry-sam-k
Copy link
Member

elnyry-sam-k commented Jul 5, 2019 •
edited

Goal:

As a ML OSS Codebase maintainer
I want to move the "transformHeaders" into a shared http library
so that it can be reused in ml-api-adapter, bulk-api-adapter, quoting-service, ALS as relevant

Tasks:

  • Move the "transformHeaders" into a shared http library
  • Create tests
  • Create Initial Release
  • Update ml-api-adapter
  • Update bulk-api-adapter
  • Update quoting-service
  • Update ALS

Acceptance Criteria:

  • "transformHeaders" functionality is moved into a shared http library
  • Bulk-api-adapter, ml-api-adapter are updated to use the above library
  • Unit tests pass (same pass rate as before)
  • Integration tests pass (same pass rate as before)

Pull Requests:

Follow-up:

Accountability:

  • Owner: TBC
  • QA/Review: TBC
@mdebarros mdebarros self-assigned this Jul 5, 2019
@ggrg ggrg mentioned this issue Jul 8, 2019
Merged
@rmothilal rmothilal assigned mdebarros and rmothilal and unassigned mdebarros Jul 12, 2019
@elnyry-sam-k elnyry-sam-k added this to the Sprint 7.1 milestone Jul 12, 2019
@elnyry-sam-k elnyry-sam-k changed the title Move the "transformHeaders" into a shared http library Move the "transformHeaders" into a shared http library -1 Jul 22, 2019
@elnyry-sam-k elnyry-sam-k mentioned this issue Jul 23, 2019
Closed
15 tasks
@elnyry-sam-k elnyry-sam-k mentioned this issue Aug 5, 2019
Closed
8 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@mdebarros mdebarros

@rmothilal rmothilal

Labels
story
Projects
None yet
Milestone
Sprint 7.1
Development

No branches or pull requests
3 participants
@elnyry-sam-k @mdebarros @rmothilal