Changed property observer to use property info instead of property name

[omerfarukz]

Bugs Fixed

#1676

PR Checklist

• [x ] Has tests (if omitted, state reason in description)
• Rebased on top of master at time of PR
• [x ] Changes adhere to coding standard

[dnfclas]

All CLA requirements met.

[brianlagunas]

This is actually much better. Sometimes it's easy to miss the most obvious approach. Have you confirmed that this works for your scenario?

[omerfarukz]

Yes, it works in my scenario. All tests are passed except DirectoryModuleCatalogFixture. Consedired it is not related with this.

[brianlagunas]

Cool. Yeah, don't worry about the DirectoryModuleCatalog tests. Those are failing because of the .NET Core 3 target. We are working on our build server right now, so as soon as we get that fixed, I'll merge.

[dansiegel]

Thanks for the PR

[lock]

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.