peopledoc / pydocusign

Python client for DocuSign signature API
Other
18 stars 21 forks source link

add, remove recipients #72

Closed smcoll closed 8 years ago

smcoll commented 8 years ago

Here is a proposal for #70. i wish that the add_envelope_recipient shortcut would return a JSON object for the one recipient, but given the way DocuSign returns a response of all recipient types, i thought that might get a little wonky.

zebuline commented 8 years ago

Nice work ! Is it possible to add some tests ?

smcoll commented 8 years ago

@zebuline yes, but not promptly. Also note, @kbussell had another approach mentioned in the thread for #70. If you prefer it, we could go that route instead.

zebuline commented 8 years ago

@smcoll I planned to release the 1.0 in few days, do you want to change something on this PR, or at least rebase it on master, just to be sure ?

zebuline commented 8 years ago

closed in favor of #93