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

#167728046 Complete user can book accommodation facility #53

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from

Conversation

richienabuk
Copy link
Collaborator

What does this PR do?

  • Implement endpoint POST /bookings for user to book an accommodation

  • Create an endpoint GET /bookings for user to see all bookings

    Description of Task to be completed?

  • Login with Supplier role

  • Create an accommodation and a room at the endpoint POST /accommodation

  • Use the new or existing room UUID to create a booking

    How should this be manually tested?

  • Clone the project and seed the database

  • Log in as a seeded user

  • Visit /bookings POST request and enter the parameters:

  • room_uuid (it must be a UUID of an existing room)

  • starting_date

  • ending_date

To see all bookings, visit GET /bookings as a logged-in user (with token)

Any background context you want to provide?

A user should be able to book accommodation at his/her destination of traveling to be assured of residence on getting there.

What are the relevant stories?

#167728046

Screenshots (if appropriate)

create booking

get all booking

Abobos and others added 2 commits September 13, 2019 02:20
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

Choose a reason for hiding this comment

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

Block must not be padded by blank lines padded-blocks

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

2 participants