Skip to content
This repository has been archived by the owner on Feb 12, 2024. It is now read-only.

Adding Gateway , based on harshjv PR #966

Merged
merged 4 commits into from
Aug 29, 2017
Merged

Conversation

ya7ya
Copy link
Contributor

@ya7ya ya7ya commented Aug 28, 2017

Hey, This is a gateway implementation based on PR #698 by harshjv . with the modifications required to accept it as mentioned in PR #698 .

Changes made:

  • rename the folder http-api to just http
  • create two folders, one being api and the other being gateway
  • No Promises library required
  • Tests

@diasdavid Please let me know if this is good or if there are any modifications required to get this PR merged. Thank you for your help.

@daviddias daviddias changed the base branch from master to feat/gateway August 29, 2017 07:23
@daviddias
Copy link
Member

@ya7ya in order to make collaboration easier, I created a new branch in the main repo and merged @harshjv PR to it so that you can target your PR to the same branch.

This is important because we want to ensure we preserve commit history (in this case, @harshjv initial contributions).

Please rebase the branch onto your PR so that there are no merge conflicts. I'll CR then :)

License: MIT
Signed-off-by: Yahya <ya7yaz@gmail.com>
License: MIT
Signed-off-by: Yahya <ya7yaz@gmail.com>
License: MIT
Signed-off-by: Yahya <ya7yaz@gmail.com>
License: MIT
Signed-off-by: Yahya <ya7yaz@gmail.com>
@ya7ya
Copy link
Contributor Author

ya7ya commented Aug 29, 2017

@diasdavid I rebased the feat/gateway branch onto the PR , Thanks for coordinating this 👍 😄

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants