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

[KeyVault] - Update KeyVault Admin samples #15613

Merged
merged 1 commit into from
Jun 9, 2021

Conversation

maorleger
Copy link
Member

This PR just updates the KeyVault admin samples to provide some context around what is happening, restore the
console logging of backup / restore results, and provide a helper method to ensure URIs are well-formed.

@maorleger maorleger requested a review from sadasant as a code owner June 8, 2021 17:50
@ghost ghost added the KeyVault label Jun 8, 2021
Copy link
Contributor

@sadasant sadasant left a comment

Choose a reason for hiding this comment

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

Looks good!

@maorleger maorleger merged commit 36d7801 into Azure:master Jun 9, 2021
@maorleger maorleger deleted the fix-admin-samples branch June 9, 2021 15:38
sadasant pushed a commit to sadasant/azure-sdk-for-js that referenced this pull request Jun 21, 2021
This PR just updates the KeyVault admin samples to provide some context around what is happening, restore the 
console logging of backup / restore results, and provide a helper method to ensure URIs are well-formed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants