search

lkaric / nestjs-twilio

Injectable Twilio client for Nestjs.
https://www.npmjs.com/package/nestjs-twilio
MIT License
42 stars 10 forks source link

cant resolve dependency tree #11

Closed rod-dd closed 3 years ago

rod-dd commented 3 years ago

Bug Report

NestJS Info

image

Describe the Bug

installing this npm causes an error

npm i nestjs-twilio

and give me this image

Expected Behavior

it should install properly or without an error.

Workaround

you can install the npm by this,

npm i nestjs-twilio  --legacy-peer-deeps

Ill update this ticket if I find something meaningful.

lkaric commented 3 years ago

@rod-dd I bumped the versions and cleaned up some stuff, can you try installing now with the latest v1.2.0, please let me know if it fixed the issue

rod-dd commented 3 years ago

@rejvban thanks man,

Managed to quick install under my project and this is what i got image

I think this "constrain" is due to this line: https://github.com/rejvban/nestjs-twilio/blob/master/package.json#L66.

Let me hear your thoughts.

lkaric commented 3 years ago

I released a v1.2.1 which hopefully supports the whole 7 major version

rod-dd commented 3 years ago

Yay! thanks man @rejvban, managed to install without an issue.

image