Skip to content
This repository was archived by the owner on Jan 23, 2025. It is now read-only.

Update docusign.js #518

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Update docusign.js
Fix for production issue getting following error and node.js crashes.

--------
/home/node/tc-api/actions/docusign.js:418
           textTabs.push({tabLabel : 'TopCoder Handle', value: user.handle});
                                                                   ^
TypeError: Cannot read property 'handle' of undefined
   at async.waterfall.envelopeId (/home/node/tc-api/actions/docusign.js:418:77)
   at fn (/home/node/tc-api/node_modules/async/lib/async.js:582:34)
   at Object._onImmediate (/home/node/tc-api/node_modules/async/lib/async.js:498:34)
   at processImmediate [as _immediateCallback] (timers.js:345:15)
  • Loading branch information
sudo-tc authored Feb 14, 2018
commit 95bf85ac3c14cda4a23b45c72e124c9c5bac86d3
6 changes: 5 additions & 1 deletion actions/docusign.js
Original file line number Diff line number Diff line change
Expand Up @@ -414,7 +414,11 @@ exports.generateDocusignViewURL = {
value: tabs[x][1]
});
}


if(!user) {
cb(new IllegalArgumentError("*********** Could not find user. *********** DOCUSIGN.JS"));
return;
}
textTabs.push({tabLabel : 'TopCoder Handle', value: user.handle});

//Prepare the POST parameters
Expand Down