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
Add Movies module #252
Add Movies module #252
Conversation
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.
Hey @devrrior Just 1 change and is all good to be Merged.
…c on one of the endpoints
hi @devrrior Your movie tests seem to be failing. :( |
|
Hi @ageddesi, I've been running the tests in multiple times and I haven't found any problem. Jest says every tests have been passed. |
Hi @devrrior I have just tried re-running and it failed again, The build issue is something that was fixed recently in dev. Do you have the latest code in this branch? |
Hey @devrrior, I have updated the branch and made the tests pass 👍 Happy hacktoberfest and thank you for all the hard work on this. |
Hi!
I added the movies module. I developed two endpoints for this module (list movies and get movie by id). Also, I added OpenApi documentation and tests for routes and utils.
Issue #206