vidly-api-node icon indicating copy to clipboard operation
vidly-api-node copied to clipboard

Upgrade vidly-api-node dependencies - July 2020

Open mattnguyengit opened this issue 4 years ago • 7 comments

  • Using Node 12
  • Newest version of all dependencies
  • Using native transaction of MongoDB (Replace fawn)
  • Fix bug if any

mattnguyengit avatar Jul 09 '20 18:07 mattnguyengit

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.

abhishekkumar08 avatar Jul 30 '20 17:07 abhishekkumar08

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

mattnguyengit avatar Jul 31 '20 07:07 mattnguyengit

Its still showing error, mind to check it again?

abhishekkumar08 avatar Aug 06 '20 06:08 abhishekkumar08

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

mattnguyengit avatar Aug 07 '20 13:08 mattnguyengit

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

amujuruprasad avatar Nov 10 '20 08:11 amujuruprasad

it worked on me. Thanks @amujuruprasad

RichmondRamil avatar Aug 07 '22 13:08 RichmondRamil

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 |

Dennis034 avatar Aug 15 '23 22:08 Dennis034