Skip to content

Comments

SDK update for Soft Delete feature#7338

Merged
isra-fel merged 2 commits intoAzure:masterfrom
SanjayHukumRana:master
Aug 23, 2019
Merged

SDK update for Soft Delete feature#7338
isra-fel merged 2 commits intoAzure:masterfrom
SanjayHukumRana:master

Conversation

@SanjayHukumRana
Copy link
Contributor

Updating SDK for soft delete properties of protected Item

Updating SDK for soft delete properties of protected Item
@azuresdkci
Copy link
Contributor

Can one of the admins verify this patch?

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.

Hello @SanjayHukumRana , please

  • follow the instructions here when generating SDK. Remember to track the .txt file, update versions and tags, etc.
  • put a link to the spec review PR in comments

@SanjayHukumRana
Copy link
Contributor Author

SanjayHukumRana commented Aug 21, 2019

rest api spec closed PR: Azure/azure-rest-api-specs#6818

From reviewer comment: follow the instructions here when generating SDK.
The link is not working.

@SanjayHukumRana SanjayHukumRana removed their assignment Aug 21, 2019
@isra-fel
Copy link
Member

isra-fel commented Aug 22, 2019

Sorry about that. Now as a member of Azure orgnization you should be able to view the link and update the PR

@isra-fel isra-fel added the Mgmt This issue is related to a management package. label Aug 22, 2019
@SanjayHukumRana
Copy link
Contributor Author

Updated PR based on the instructions here.

@isra-fel isra-fel merged commit 62c7b78 into Azure:master Aug 23, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Mgmt This issue is related to a management package.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants