Skip to content
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

Input value not required for start_new (and call_activity) #93

Merged
merged 8 commits into from
Mar 27, 2020

Conversation

davidmrdavid
Copy link
Collaborator

@davidmrdavid davidmrdavid commented Mar 26, 2020

This PR adds default None values for instance_id and input to client.start_new, making our library's API a little cleaner. Samples were updating accordingly.

It addresses: #91

I did not make any changes to call_activity as that seemed to already have default values. Please confirm

@davidmrdavid
Copy link
Collaborator Author

I seemed to have pushed a few too many things at once. My mistake, Let me fix this real quick

@davidmrdavid
Copy link
Collaborator Author

Ok, fixed now :)
Please review at when you get the chance

@davidmrdavid davidmrdavid merged commit 366aab0 into dev Mar 27, 2020
@davidmrdavid davidmrdavid deleted the djusto/do-not-require-inputs branch March 27, 2020 04:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants