-
Notifications
You must be signed in to change notification settings - Fork 54
Apiv2 #3
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Apiv2 #3
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think we should fix some of the inconsistencies but it's overall looking pretty good
app/templates/home.html
Outdated
<h4 id="example020">20. <a href="eg020">Send an envelope with SMS Recipient Authentication</a></h4> | ||
<p> | ||
Anchor text | ||
(<a target='_blank' href="https://support.docusign.com/en/guides/AutoPlace-New-DocuSign-Experience">AutoPlace</a>) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
is this the right article for this example?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
added details to describe each added example on the template home.html file.
(<a href="https://support.docusign.com/en/guides/AutoPlace-New-DocuSign-Experience">AutoPlace</a>) | ||
is used to position the signing fields in the documents. | ||
</p> | ||
<p>This is a general example of creating and sending an envelope (a signing request) to a recipient that requires an Access Code to open</p> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"... open it"?
@MattKingDS can you please review this as well? |
adds in updated readme, views, and controller code for v2 methods.