Skip to content

Commit e9a8613

Browse files
authored
Merge pull request #178 from docusign/fix-updated-maestro-to-work-with-new-sdk
Updated trigger workflow method and base url for Meastro
2 parents 5a7323f + 0efffaf commit e9a8613

File tree

5 files changed

+36
-32
lines changed

5 files changed

+36
-32
lines changed

config/appsettings.example.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,7 @@
2929
"clickAPIUrl": "https://demo.docusign.net/clickapi",
3030
"adminAPIUrl": "https://api-d.docusign.net/management",
3131
"monitorApiUrl": "https://lens-d.docusign.net",
32-
"maestroApiUrl": "https://demo.services.docusign.net",
32+
"maestroApiUrl": "https://apps-d.docusign.com/api/maestro",
3333
"webformsApiUrl": "https://apps-d.docusign.com/api/webforms/v1.1",
3434
"codeExamplesManifest": "https://raw.githubusercontent.com/docusign/code-examples-csharp/master/manifest/CodeExamplesManifest.json"
3535
}

lib/maestro/examples/triggerWorkflow.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -58,7 +58,7 @@ const triggerWorkflow = async (workflow, args) => {
5858
//ds-snippet-end:Maestro1Step4
5959

6060
//ds-snippet-start:Maestro1Step5
61-
const triggerResponse = await workflowTriggerApi.triggerWorkflow(triggerPayload, args.accountId, { mtid, mtsec });
61+
const triggerResponse = await workflowTriggerApi.triggerWorkflow(triggerPayload, args.accountId, args.workflowId, { mtid, mtsec });
6262

6363
return triggerResponse;
6464
//ds-snippet-end:Maestro1Step5

lib/maestro/workflowUtils.js

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -591,9 +591,10 @@ async function publishWorkflow(args, workflowId) {
591591
try {
592592
await workflowManagementApi.publishOrUnPublishWorkflowDefinition(new docusign.DeployRequest(), args.accountId, workflowId);
593593
} catch (error) {
594-
const isConsentRequired = error?.response?.body?.message?.toLowerCase() === 'consent required';
594+
const isConsentRequired = error?.response?.body?.message?.toLowerCase() === 'consent required'
595+
|| error?.body?.message?.toLowerCase() === 'consent required';
595596
if (isConsentRequired) {
596-
return error.response.body.consentUrl;
597+
return error?.response?.body?.consentUrl || error?.body?.consentUrl;
597598
} else {
598599
throw error;
599600
}

package-lock.json

Lines changed: 30 additions & 27 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -36,7 +36,7 @@
3636
"docusign-admin": "^2.3.0",
3737
"docusign-click": "^2.2.0",
3838
"docusign-esign": "^7.0.0-rc1",
39-
"docusign-maestro": "1.0.0-rc3",
39+
"docusign-maestro": "1.0.0-rc5",
4040
"docusign-monitor": "^2.1.0",
4141
"docusign-rooms": "^2.0.0",
4242
"docusign-webforms": "^1.0.0",

0 commit comments

Comments
 (0)