lkaric / nestjs-twilio

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

Constructor injection gives the following error #3

Closed codebanesr closed 4 years ago

codebanesr commented 4 years ago

'TwilioClient' refers to a value, but is being used as a type here. Did you mean 'typeof TwilioClient'?ts(2749) Parameter 'client' of constructor from exported class has or is using private name 'TwilioClient'.ts(4063)

Ubuntu 20.10 Nestjs


  constructor(
    @InjectTwilio() 
    private readonly client: TwilioClient
  ) {}
lkaric commented 4 years ago

@shanurrahman Sorry for the inconvenience, the issue is resolved.

codebanesr commented 4 years ago

Thanks

On Mon, Sep 21, 2020 at 1:42 AM Lazar Karić notifications@github.com wrote:

@shanurrahman https://github.com/shanurrahman Sorry for the inconvenience, the issue is resolved.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/rejvban/nestjs-twilio/issues/3#issuecomment-695830608, or unsubscribe https://github.com/notifications/unsubscribe-auth/AEI5ZGXLT3BNGJUIL4BR3G3SGZOZPANCNFSM4RTN76CA .

abdallahbedir2 commented 6 months ago

Why InjectTwilio is no longer exported?

lkaric commented 6 months ago

@abdallahbedir2 Because of the architectural changes and introduction of the configurable module builder. It should still be available with older versions, but I do not provide any support for legacy syntax.

Any specific use case that cannot be done with the standard (current) approach? If so, please open a new issue with the minimal reproduction.