dsccommunity / xRemoteDesktopSessionHost

This module contains DSC resources for the management and configuration of Microsoft Remote Desktop Session Host (RDSH).
MIT License
36 stars 47 forks source link

xRDRemoteApp - Change datatype for UserGroups parameter from [string] to [string[]] #22

Closed stiwenn closed 6 years ago

stiwenn commented 7 years ago

Update of MSFT_xRDRemoteApp.psm1 and MSFT_xRDRemoteApp.schema.mof in one PR.

This PR should contain 2 files changes executed previously in separate PR's: https://github.com/PowerShell/xRemoteDesktopSessionHost/pull/18 https://github.com/PowerShell/xRemoteDesktopSessionHost/pull/19


This change is Reviewable

bgelens commented 7 years ago

Thanks @stiwenn! 👍 Please sign the contribution license so I can accept the PR. I'll start my review shortly

bgelens commented 7 years ago

@stiwenn, Could you update the Readme unreleased and xRDRemoteApp sections. And sign the CLA please :smile:


Reviewed 2 of 2 files at r1. Review status: all files reviewed at latest revision, 1 unresolved discussion.


DSCResources/MSFT_xRDRemoteApp/MSFT_xRDRemoteApp.schema.mof, line 15 at r1 (raw file):

    [write, Description("Specifies the index within the icon file (specified by the IconPath parameter) where the RemoteApp program's icon can be found.")] uint32 IconIndex;
    [write, Description("Specifies the path to a file containing the icon to display for the RemoteApp program identified by the Alias parameter.")] string IconPath;
    [write, Description("Specifies a domain group that can view the RemoteApp in RD Web Access, and in RemoteApp and Desktop Connections. To allow all users to see a RemoteApp program, provide a value of Null.")] string UserGroups[];

Please update the description to something like, Specifies a or multiple domain group(s) or simular


Comments from Reviewable

johlju commented 6 years ago

@stiwenn This PR is still waiting for the CLA to be signed. Do you want to sign the CLA? If you know you have signed the CLA already then let me know. To sign the CLA, please click Details on this row.

image

johlju commented 6 years ago

Closing this since CLA has not been signed. @stiwenn if you want to continue work on this, please reopen and sign the CLA.