adopted a few stylistic suggestions from CJ.

docusign · Oct 28, 2015 · d75ff66 · d75ff66
1 parent afbef66
commit d75ff66
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/components/envelopes.js b/components/envelopes.js
@@ -859,6 +859,7 @@ function getTemplateView (apiToken, baseUrl, templateId, returnUrl) {
  * @returns {Promise} - A thenable bluebird Promise fulfilled with the list of recipients
  */
 function getRecipients (apiToken, baseUrl, envelopeId, include_tabs) {
+  include_tabs = include_tabs != null ? include_tabs : false;
   var options = {
     method: 'GET',
     url: baseUrl + '/envelopes/' + envelopeId + '/recipients?include_tabs=' + include_tabs,

diff --git a/test/component-tests/envelopes.spec.js b/test/component-tests/envelopes.spec.js
@@ -16,6 +16,7 @@ let roleName = config.templateRole;
 let clientUserId = '1001';
 let returnUrl = 'http://www.docusign.com/devcenter';
 let emailSubject = 'DocuSign API - Signature Request on Document Call';
+let include_tabs = true;
 let additionalParams = {};
 let recipients = {
   signers: [{
@@ -135,7 +136,7 @@ test(function getRecipients (t) {
 });
 
 test(function getRecipientsWithTabs (t) {
-  return client.envelopes.getRecipients(envelopeId, true)
+  return client.envelopes.getRecipients(envelopeId, include_tabs)
   .then(response => {
     t.ok(response.signers);
     t.ok(response.signers[0].isBulkRecipient);