-
Notifications
You must be signed in to change notification settings - Fork 7
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
#167727662-Re-Confirm-Travel-Requests #47
Conversation
b3abbfb
to
04bd48e
Compare
04bd48e
to
257326d
Compare
79ed639
to
e3ea728
Compare
e3ea728
to
77a0ec7
Compare
Please, re-implement this feature. create a separate route handler for this feature in implement the logic for this feature in the route handler you will create. |
6c77078
to
6c80f92
Compare
2caf5eb
to
d5e3505
Compare
7c8c877
to
0ecc161
Compare
* feature(request):add multi-city request route Create feature to confirm approved travel requests * feature(request):add multi-city request route
0ecc161
to
189c013
Compare
92a1e31
to
4920eb6
Compare
…ub.com/andela/rogue-backend into ft-reconfirm-travel-requests-167727662
4920eb6
to
eefce86
Compare
'Could not confirm the request. Please try again', 400); | ||
} | ||
return HelperMethods.clientError(res, | ||
'The request you are trying to confirm cannot be found', 404); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Expected a newline before ')' function-paren-newline
if (approvedRequest) { | ||
if (approvedRequest.status === 'confirmed') { | ||
return HelperMethods.clientError(res, | ||
'This request has already been confirmed', 404); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Expected a newline before ')' function-paren-newline
}); | ||
if (approvedRequest) { | ||
if (approvedRequest.status === 'confirmed') { | ||
return HelperMethods.clientError(res, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Expected a newline after '(' function-paren-newline
What does this PR do
It implements a confirmation feature for a line manager to confirm an approved travel request.
Description of Task to be completed?
How should this be manually tested?
cd
into the project directoryyarn install
to install the project's dependencyyarn start
to start the appWhat are the relevant pivotal tracker stories?
#167727662
Any background context you want to add?