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

fix(azure-iothub): schedule deviceMethod and updateTwin need to speci… #854

Merged
merged 1 commit into from
Jul 20, 2020

Conversation

anthonyvercolano
Copy link
Contributor

…fy defaults when arguments not supplied

Both methods were sending null for the values if the arguments to the methods were not supplied.
The service requires actual values. Supplied Date.now and 3600 as defaults.

fix #847

…fy defaults when arguments not supplied

Both methods were sending null for the values if the arguments to the methods were not supplied.
The service requires actual values.  Supplied Date.now and 3600 as defaults.

fix Azure#847
Copy link
Contributor

@zolvarga zolvarga left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:shipit:

@anthonyvercolano anthonyvercolano merged commit 10284ea into Azure:master Jul 20, 2020
@anthonyvercolano anthonyvercolano deleted the repro-issue-847 branch July 20, 2020 21:12
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.

Schedule job only works with all parameters given
2 participants