docusign-esign-node-client icon indicating copy to clipboard operation
docusign-esign-node-client copied to clipboard

phoneNumber for Signer parameter is not in @types package

Open peiofour opened this issue 2 years ago • 8 comments

Hello DocuSign,

I work on a project using the docusign-esign 5.20.0 npm package and its types package 5.19.0. Using the Signer class, I noticed that my linter doesn't accept the phoneNumber parameter. I checked on the types docusign-esign package, it is not in the Signer interface.

image

Is the types package not updated or is the parameter deprecated?

Thanks in advance!

peiofour avatar Jan 17 '23 13:01 peiofour

This is the old way to do this, this has moved under workflow. See full code in https://github.com/docusign/code-examples-node/blob/master/lib/eSignature/examples/phoneAuthentication.js


  let signer1 = docusign.Signer.constructFromObject({
    email: args.signerEmail,
    name: args.signerName,
    recipientId: RecipientId,
    routingOrder: "1",
    deliveryMethod: "Email",
    tabs: signer1Tabs,
    identityVerification: { workflowId: args.workflowId, steps: null, "inputOptions":[{"name":"phone_number_list","valueType":"PhoneNumberList","phoneNumberList":[{"countryCode":args.countryCode,"code":"1","number":args.phoneNumber}]}], "idCheckConfigurationName":""}
  });

InbarGazit avatar Jan 17 '23 17:01 InbarGazit

So the doc needs to be updated: https://developers.docusign.com/docs/esign-rest-api/how-to/request-signature-sms/

peiofour avatar Jan 18 '23 10:01 peiofour

@peiofour for the most up-to-date reference for this SDK, please refer to http://docusign.github.io/docusign-esign-node-client. These get automatically updated during our release process and will be the most consistent reference to the SDK

acooper4960 avatar Jan 24 '23 20:01 acooper4960

@peiofour I'm confused, is this about SMS delivery or SMS authentication? these are two different things.

InbarGazit avatar Jan 24 '23 22:01 InbarGazit

@InbarGazit It's for SMS delivery (https://developers.docusign.com/docs/esign-rest-api/esign101/concepts/sms-delivery/)

peiofour avatar Jan 30 '23 09:01 peiofour

OK, thanks, do you still have an issue? or can I close this out?

InbarGazit avatar Jan 30 '23 23:01 InbarGazit

Yes ! The issue didn't change, tslint does'nt accept phoneNumber parameter as we can see here : https://developers.docusign.com/docs/esign-rest-api/esign101/concepts/sms-delivery/

image

peiofour avatar Feb 01 '23 09:02 peiofour

And there is no additionalNotifications field for the Signer type Screenshot 2024-09-04 at 13 56 57 Screenshot 2024-09-04 at 13 57 26

Olexandr-Liaguta avatar Sep 04 '24 11:09 Olexandr-Liaguta