-
Notifications
You must be signed in to change notification settings - Fork 59
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
Get-IssuedRequest implement order by property parameter #88
Comments
I believe, that default sorting should be ascending. At least, it is convenient. |
Dear Vadims, |
is it common? I personally, rarely use sorting in MMC.
Relying on RequestID may have side effects and works stable only for pending requests. For other cert states this may not work. When certificate request is submitted, a unique request ID is associated with particular request entry. And when request state is changed (issued, revoked, denied) its ID is not changed, thus after changing the state won't place request as topmost entry in particular folder in all cases. Thus, different columns should be used by default in every request category to reflect latest actions. |
I'll avoid using MMC, as it is slow, but yes, usually I sort it first using RequestID. knowing it might not be the most ideal start. You're right, depending what you are looking for, different attributes are off interest, and even exist. Now, nothing prevents using different attribute for sorting for different views:
|
Hello Vadims is there an intend to work on this request? |
Not yet. I will pass this to next major update. I spent too much time on OCSP and DCOM stuff. It is till in my todo list. |
thank you for your quick feedback. |
Hi Vadims
as discussed, I propose to implement a new parameter to allow for sorting.
-OrderBy
defines the property used to sort. Only some attributes are possible, specified as enum. Default order by descending, default property RequestID
-Ascending
change the order to be Descending.
-Descending
may not be required if default is assumed.
Alternative:
-OrderByDescending
Sets the order and property. An alias "-Descending" should be allowed.
-OrderByAscending
Sets the order and property. An alias "-Ascending" should be allowed.
The text was updated successfully, but these errors were encountered: