-
Notifications
You must be signed in to change notification settings - Fork 87
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
PR for a bunch of open issues #687
Conversation
@@ -15,24 +15,34 @@ function Get-RubrikVolumeGroup | |||
.LINK | |||
https://rubrik.gitbook.io/rubrik-sdk-for-powershell/command-documentation/reference/get-rubrikvolumegroup | |||
|
|||
.EXAMPLE | |||
Get-RubrikVolumeGroup -Name 'Quokka volumes' |
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.
👀
|
||
.EXAMPLE | ||
$currentreport = Get-RubrikReport -name BoringReportName -DetailedObject | ||
$currentreport.name = "Jaap's QuokkaReport" |
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.
👀 👀
.EXAMPLE | ||
Get-RubrikReport -Name 'Boring Report' -DetailedObject | Set-RubrikReport -NewName 'Quokka Report' | ||
|
||
This will rename the report named 'Boring Report' to 'Quokka Report' |
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.
👀 👀 👀
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.
Most impressive
@jaapbrasser @shamsway |
Description
This PR fixes pretty much everything that has been reported broken. There's a lot of changes so I don't envy whoever made the mistake of picking this up for review.
Related Issue
Resolves #688
Resolves #681
Resolves #679
Resolves #671
Resolves #662
Resolves #660
Resolves #659
Resolves #655
Resolves #654
Resolves #653
Resolves #652
Resolves #651
Resolves #647
Motivation and Context
It solves all problems.
How Has This Been Tested?
Unit tests still pass (except the vfg one, but that has been fixed in the other PR)
Screenshots (if appropriate):
Types of changes
What types of changes does your code introduce? Put an
x
in all the boxes that apply:Checklist:
Go over all the following points, and put an
x
in all the boxes that apply. If you're unsure about any of these, don't hesitate to ask. We're here to help!