-
Notifications
You must be signed in to change notification settings - Fork 4.8k
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
[Logic] Sync SDK with swagger updates #5032
[Logic] Sync SDK with swagger updates #5032
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.
Please pull latest changes from psSdkJson6, run git clean -xdff
, msbuild build.proj
and regenerate the code.
Add 2 new endpoints to the SDK. Also fix a bug that caused a workflow with an integration account to be unable to be updated
1828a65
to
4330753
Compare
Regenerated the code from a fresh pull. |
@@ -8,9 +8,14 @@ | |||
<TreatWarningsAsErrors>true</TreatWarningsAsErrors> | |||
<Version>1.0.0</Version> | |||
</PropertyGroup> | |||
<ItemGroup> |
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.
@refortie remove line 11-16 you don't need it. (nit)
Please remove this next time.
Description
Updating the Logic Apps SDK based on the newest version of the swagger. This adds 2 new endpoints which both have Get and List. Also this includes a bugfix where a user could not update a Logic App that had an attached Integration Account.
Bugfix PR
New endpoints PR
This checklist is used to make sure that common guidelines for a pull request are followed.
General Guidelines
Testing Guidelines
SDK Generation Guidelines
*.csproj
andAssemblyInfo.cs
files have been updated with the new version of the SDK.