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

#167728041 Travel admins should be able to approve accommodation facility #56

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

Conversation

dieudonneAwa
Copy link
Collaborator

What does this PR do?

  • Gets all unapproved accommodation facilities
  • Approves an accommodation facility

Description of Task to be completed?
Have the following endpoints working:

  • GET /api/v1/admin/accommodation
  • PUT /api/v1/admin/approve_accommodation

How should this be manually tested?
After cloning the repo,

  • run npm i, then npm run dev
  • For each endpoint,
    Sample request bodies
  1. GET /api/v1/admin/accommodation

  2. PUT /api/v1/admin/approve_accommodation

{
  "accommodation_uuid": "uuid of any facility"
}

Any background context you want to provide?

  • None

What are the relevant stories?

Screenshots (if appropriate)
Screenshot (103)
Screenshot (105)

Questions:

  • None

dieudonneAwa and others added 10 commits August 27, 2019 14:25
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]
create endpoint for getting all unapproved facilities
create endpoint for approving facilities

[Finishes #167728041]
@dieudonneAwa dieudonneAwa changed the title #167728041 Travel should be able to approve accommodation facility #167728041 Travel admins should be able to approve accommodation facility Oct 13, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants