-
Notifications
You must be signed in to change notification settings - Fork 1
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
#170180280 Manager get all and specific user booking Requests #48
#170180280 Manager get all and specific user booking Requests #48
Conversation
8c086d8
to
677ad56
Compare
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.
Well done @william000000. I liked how you made it so simple to display all bookings. I request you add more test.
res.should.have.status(404); | ||
res.body.should.be.an('object'); |
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.
Add more test.
res.should.have.status(200); | ||
res.body.should.be.an('object'); |
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.
Please add more test.
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.
Thank you @NiyoEric, I have already implemented
677ad56
to
bc6f186
Compare
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.
Nice job @william000000
Just have a look at some feedback I left
bc6f186
to
fbc29ab
Compare
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.
Nice work @william000000 .
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.
Hi @william000000 , nice job.
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.
Great job @william000000 Looks good to me!
…quest made - Get specific booking an accommodation request - Fix the issue where user was able to book an accommodation facility that s/he is not the owner - Test for get user booking request - Add swagger documentation [Finishes #170180280]
fbc29ab
to
e7dbed9
Compare
What does this PR do?
Manager get specific or all booking accommodation requests
Description of Task to be completed?
userId
How should this be manually tested?
git clone https://github.com/andela/team-odd-bn-backend.git
cd project-folder
and runnpm i
to install all dependenciesgit checkout ft-get-all-and-specific-booking-request-170180280
npx sequelize-cli db:seed:all
npm run dev-start
Any background context you want to provide?
N/A
What are the relevant pivotal tracker stories?
#170180280
Screenshots (if appropriate)
Get all users' booking requests
Get specific user booking requests
Questions:
N/A