mosh-hamedani / vidly-api-node

309 stars 285 forks source link

Upgrade vidly-api-node dependencies - July 2020 #33

Open mattnguyengit opened 4 years ago

mattnguyengit commented 4 years ago
abhishekkumar08 commented 3 years ago

Kindly reply if I can work on this issue I have solved it on my system as the jest dependency is old and it create an error when running the command node index.js. So the dependency needs to be uninstalled and then install again the latest one therefore it workks ok. And there are some code changes which have to be done as the previous codes are not matching up with the tasks.

mattnguyengit commented 3 years ago

Thank you for reply.

I updated a PR for all of it: https://github.com/mosh-hamedani/vidly-api-node/pull/34

You can review it. Some jest issues I handled it.

Updated: @hapi/joi is backed to joi. But the changes on my code is still valid. Just change @hapi/joi to joi

abhishekkumar08 commented 3 years ago

Its still showing error, mind to check it again?

mattnguyengit commented 3 years ago

If you have error can't handle by yourself. Don't hesitate to ask mosh's forum: https://forum.codewithmosh.com/

amujuruprasad commented 3 years ago

https://github.com/amujuruprasad/vidly-api-nodejs

RichmondRamil commented 1 year ago

it worked on me. Thanks @amujuruprasad

Dennis034 commented 10 months ago

FAIL tests/unit/middleware/auth.test.js ● Test suite failed to run

pls need help for dis .....Cannot find module '../../models/users' from 'tests/unit/middleware/auth.test.js'

> 1 | const {Users} = require('../../models/users');
    |                 ^
  2 | const auth = require('../../middleware/auth');
  3 | const mongoose = require('mongoose');
  4 |