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

#167749962 Implement user can view all own travel requests submitted #28

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

Conversation

ldonjibson
Copy link
Collaborator

What does this PR do?
This PR implements user can view all own travel requests submitted

Description of Tasks to be completed

  • Create new request controller to retrieve all requests of a user
  • Create new service to retrieve the data needed from database after
    authentication has passed
  • Create route to access and retreive databse response
  • Create test for the endpoint
  • Add endpoint to swagger documentation

How should this be manually tested?

Relevant pivotal tracker story
#167749962

@ldonjibson ldonjibson changed the title #167749962 Implement user can view all own requests submitted #167749962 Implement user can view all own travel requests submitted Sep 10, 2019
@ldonjibson ldonjibson force-pushed the ft-all-requests-submited-by-user-167749962 branch from 12b3c98 to 65610e7 Compare September 12, 2019 17:37
src/services/RequestService.js Outdated Show resolved Hide resolved
src/controllers/RequestController.js Outdated Show resolved Hide resolved
src/controllers/RequestController.js Outdated Show resolved Hide resolved
src/controllers/RequestController.js Outdated Show resolved Hide resolved
src/controllers/RequestController.js Outdated Show resolved Hide resolved
src/controllers/RequestController.js Outdated Show resolved Hide resolved
src/controllers/RequestController.js Outdated Show resolved Hide resolved
src/controllers/RequestController.js Outdated Show resolved Hide resolved
src/controllers/RequestController.js Outdated Show resolved Hide resolved
src/controllers/RequestController.js Outdated Show resolved Hide resolved
@ldonjibson ldonjibson force-pushed the ft-all-requests-submited-by-user-167749962 branch 4 times, most recently from 2ede01e to d1c5da9 Compare September 12, 2019 20:51
src/database/seeders/20190911231139-request.js Outdated Show resolved Hide resolved
src/controllers/RequestController.js Outdated Show resolved Hide resolved
src/controllers/RequestController.js Outdated Show resolved Hide resolved
@ldonjibson ldonjibson force-pushed the ft-all-requests-submited-by-user-167749962 branch 4 times, most recently from 4bfbd76 to b7a21f0 Compare September 12, 2019 21:32
src/controllers/RequestController.js Outdated Show resolved Hide resolved
src/controllers/RequestController.js Outdated Show resolved Hide resolved
src/controllers/RequestController.js Outdated Show resolved Hide resolved
@ldonjibson ldonjibson force-pushed the ft-all-requests-submited-by-user-167749962 branch from b7a21f0 to 26bbd89 Compare September 12, 2019 21:34
@ldonjibson ldonjibson force-pushed the ft-all-requests-submited-by-user-167749962 branch from 26bbd89 to 73002de Compare September 12, 2019 21:37
src/tests/request.test.js Outdated Show resolved Hide resolved
@ldonjibson ldonjibson force-pushed the ft-all-requests-submited-by-user-167749962 branch 2 times, most recently from c302e78 to 815d6eb Compare September 12, 2019 21:56
@ldonjibson ldonjibson force-pushed the ft-all-requests-submited-by-user-167749962 branch 2 times, most recently from f262c40 to b68a62c Compare September 12, 2019 22:17
}
Responses.setSuccess(200, 'Trip requests retrieved successfully', userTrips);
return Responses.send(res);
})
Copy link

Choose a reason for hiding this comment

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

Missing semicolon semi

Copy link
Collaborator

Choose a reason for hiding this comment

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

Kindly add semi colon to line 56

- create new request controller to retrieve all requests of a user
- create new service to retrieve the data needed from database after
  authentication has passed
- create route to access and retreive databse response
- create test for the endpoint
- add endpoint to swagger documentation

[Delivers #167749962]
@ldonjibson ldonjibson force-pushed the ft-all-requests-submited-by-user-167749962 branch from b68a62c to 3d1712e Compare September 12, 2019 22:20
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.

2 participants