Skip to content

ApiManagement - new verison 2020-12-01 update #15524

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

Merged
merged 17 commits into from
Aug 4, 2021

Conversation

KacieKK
Copy link
Contributor

@KacieKK KacieKK commented Jul 23, 2021

Description

update to latest SDK

Checklist

@KacieKK KacieKK requested a review from solankisamir July 23, 2021 00:05
@isra-fel isra-fel self-assigned this Jul 23, 2021
Copy link
Member

@isra-fel isra-fel left a comment

Choose a reason for hiding this comment

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

As the log of CI suggests, please fix 2 issues in help documents

Target      : New-AzApiManagementKeyVaultObject.md
  Description : Online version in the header of the file is incorrect. The 
                corresponding URL should be the following: https://docs.microsoft
                .com/powershell/module/az.apimanagement/new-azapimanagementkeyvau
                ltobject
  
  Target      : Sync-AzApiManagementKeyVaultSecret.md
  Description : Online version in the header of the file is incorrect. The 
                corresponding URL should be the following: https://docs.microsoft
                .com/powershell/module/az.apimanagement/sync-azapimanagementkeyva
                ultsecret

@isra-fel
Copy link
Member

Besides, some scenario tests are failing

@solankisamir
Copy link
Member

/azp run

@azure-pipelines
Copy link
Contributor

Commenter does not have sufficient privileges for PR 15524 in repo Azure/azure-powershell

@KacieKK KacieKK removed their assignment Jul 30, 2021
@KacieKK
Copy link
Contributor Author

KacieKK commented Jul 30, 2021

@isra-fel Please take a look again

@KacieKK KacieKK requested a review from isra-fel August 2, 2021 18:11
@solankisamir
Copy link
Member

@isra-fel can you please take a look at this?

@VeryEarly VeryEarly assigned VeryEarly and unassigned isra-fel Aug 4, 2021
@VeryEarly VeryEarly merged commit dcc2e4b into Azure:main Aug 4, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants