megel / powerapps-helper

MIT License
27 stars 3 forks source link

Fix environment comparison case sensitivity and update Microsoft.PowerApps.CLI to 1.8.5 #10

Closed ikappas closed 3 years ago

ikappas commented 3 years ago

Hello Michael,

Thank you for your efforts. We have run into a bug where existing published PowerApps are not being displayed under the PowerApps node.

The Get-Apps API call returns the results as expected but are subsequently filtered out due to an environment.id mismatched comparison because of case sensitivity. (Default vs default). This PR fixes the issue by lowercasing the comparison.

Also, webpack requirements in the gulpfile were commented out since they are not used and raise an error.

Lastly, the PowerApps.CLI nuget package was updated to the latest version 1.8.5.