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.

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

#167728045 Implements suppliers/hosts should create accommodation facility #52

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

Abobos
Copy link
Collaborator

@Abobos Abobos commented Sep 13, 2019

What does this PR do?

Implement user can make a return trip request

Description of Task to be completed?

  • Create endpoint to post accommodation faciltiy and room
  • Create endpoint to post room for a particular accommodation
  • Create endpoint to get all accommodation facility posted by suppliers
  • Create endpoint to get a particular accommodation facility and all
    available rooms

How should this be manually tested?

Any background context you want to provide?

None

What are the relevant stories?
#167728025

Screenshots (if appropriate)
Postman 9_13_2019 2_00_08 AM
Postman 9_13_2019 2_02_02 AM
Postman 9_13_2019 2_03_04 AM
Postman 9_13_2019 2_02_21 AM

Questions:
None

tests/accommodation.test.js Outdated Show resolved Hide resolved
tests/accommodation.test.js Outdated Show resolved Hide resolved
tests/accommodation.test.js Outdated Show resolved Hide resolved
facility

- Create endpoint to post accommodation faciltiy and room
- Create endpoint to post room for a particular accommodation
- Create endpoint to get all accommodation facility posted by suppliers
- Create endpoint to get a particular accommodation facility and all
available rooms

[Finishes #167728045]
Copy link
Collaborator

@dieudonneAwa dieudonneAwa left a comment

Choose a reason for hiding this comment

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

LGTM

@Wokoro
Copy link
Collaborator

Wokoro commented Sep 14, 2019

Nice work man.

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

Successfully merging this pull request may close these issues.

None yet

4 participants