Reggie Gibson

@mhendric Looks like we're missing some documentation on how to do this. Here's an example on how to manage multiple log custom fields with xWebSite: ``` configuration websitelog { Import-DscResource...

Hi @OffColour, it depends on which resource you are using and if you're targeting a specific website or applying at the server level. If configuring log custom fields at the...

Hi @gstorme - here's some info on how to pick up abandoned work: https://dsccommunity.org/guidelines/contributing/#how-to-continue-working-on-a-pull-request-pr-when-an-author-contributor-is-unable-to-complete-it

Also, before we get too far in the review, we don't need to keep the 'x' prefix in the name. Similar to the latest resource added to this module, WebApplicationHandler,...

Hi @t3mi , nice work on this! Can we separate the changes into two PRs? One for the new FTP resource and another for the changes to xWebsite and xWebApplication?...

Oh I see, you consolidated reusable functions for this new resource. Thanks for that! Can you describe the breaking change a little more? If we're going to include a breaking...

Hopefully I will get back to this over the weekend to provide feedback. Anyone is welcome to help review in the meantime.

Hello @SergeDubovsky. Anytime you need to set a web configuration property using a hash table, that indicates it is a collection, which would normally require **xWebConfigPropertyCollection** to manage it. In...

@johlju Yes, after testing some more I can see it is not going to work as-is with xWebConfigProperty. Since this is a collection with a single property that we're trying...

I haven't put much thought into a potential solution yet, but yes, that makes the most sense to me.