Skip to content

Fixes for Azure Automation module#6

Closed
Francisco-Gamino wants to merge 1 commit intosafeermohammed:previewfrom
Francisco-Gamino:Automation-fixes
Closed

Fixes for Azure Automation module#6
Francisco-Gamino wants to merge 1 commit intosafeermohammed:previewfrom
Francisco-Gamino:Automation-fixes

Conversation

@Francisco-Gamino
Copy link
Copy Markdown
Collaborator

This PR contains the following fixes:

  • Fixing exception handling for source control and webhook cmdlets
  • Fixing odata filter for getting a webhook by runbook name
  • Fixing JobSchedule by setting the correct JobScheduleId.

…ing odata filter for getting a webhook by runbook name; Fixing JobSchedule by setting the correct JobScheduleId.
}
}
else
catch (ErrorResponseException ex)
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

There should be common class that handles this. Is it not working?

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Check ExecuteCmdlet in Base class

@Francisco-Gamino
Copy link
Copy Markdown
Collaborator Author

This PR has been replaced by #7. Closing PR.

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.

3 participants