Skip to content

Conversation

@hiaga
Copy link
Member

@hiaga hiaga commented May 13, 2022

Description

Added MUA support for V1 vaults

Checklist

  • I have read the Submitting Changes section of CONTRIBUTING.md
  • The title of the PR is clear and informative
  • The appropriate ChangeLog.md file(s) has been updated:
    • For any service, the ChangeLog.md file can be found at src/{{SERVICE}}/{{SERVICE}}/ChangeLog.md
    • A snippet outlining the change(s) made in the PR should be written under the ## Upcoming Release header -- no new version header should be added
  • The PR does not introduce breaking changes
  • If applicable, the changes made in the PR have proper test coverage
  • For public API changes to cmdlets:

Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
8 Added support for Multi-user authorization using Resource Guard for Recovery sevrices vault.
* Added support for Multi-user authorization using Resource Guard for Recovery sevrices vault.

Copy link
Member Author

Choose a reason for hiding this comment

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

typo, will update

@wyunchi-ms
Copy link
Contributor

Since you update the SDK version, you may need to re-record the test cases.

@hiaga hiaga force-pushed the hiaga/MUA branch 3 times, most recently from 53ce9cd to 4c7271a Compare May 17, 2022 14:31
@wyunchi-ms wyunchi-ms changed the base branch from main to release-2022-05-24 May 18, 2022 01:03
wyunchi-ms
wyunchi-ms previously approved these changes May 18, 2022
Added cross subscription restore

updated help text

Re-recorded test cases
@wyunchi-ms
Copy link
Contributor

/azp run azure-powershell - security-tools

@azure-pipelines
Copy link
Contributor

Azure Pipelines successfully started running 1 pipeline(s).

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.

2 participants