marclelijveld / External-Tools-Model-Documentation

This repository includes everything that is needed in order to run the External Tools capability in Power BI Desktop and generate Model Documentation.
https://data-marc.com/model-documenter/
MIT License
110 stars 42 forks source link

Suggestion: Powershell Execution Policy Bypass added to 051-Data-Marc_ModelDocumentor.pbitool.json #19

Closed paul-rami closed 2 years ago

paul-rami commented 3 years ago

Hi Marc,

First of all thanks for this amazing tool!

Just ran into the 3rd issue described on the FAQ, where the PowerShell window pops up but no action happens afterward.

I am not a developer so I don't feel comfortable playing with the PowerSheel configuration, and wondered if adding the following line (see image below) would work in bypassing the execution policies? As you see just picked the idea from another external tool that works with my current setup (Hot Swap Connections).

Thanks again for your time!

image

paul-rami commented 3 years ago

Hi Marc, I tried adding that line to "051-Data-Marc_ModelDocumentor.pbitool.json" with no luck. Do you have any recommendation that does not involve digging too deep in PowerShell policies?

marclelijveld commented 2 years ago

Hi @paul-rami ,

Thanks for bringing this to my attention and my apologies for this horrible late reply! I see you explored other options to bypass the execution policies. Did you get this to work? On my end, I will reach out to the developer of hot swap connections and ask how exactly this works, so I can potentially include this in the future version of the tool.

Thanks for now! --Marc

marclelijveld commented 2 years ago

Hi @paul-rami,

I have been in contact with Steve Campbell. He is the developer of the Hot Swap Connections external tool. He told me that it is due to a change they made for bus ops tool - they move the location of the powershell scripts. As a result, the filename broke as it had spaces. This was fixed on the new version of bus ops and I made sure your tool was too.

How to fix:

Kudos for this solution all to @PowerBISteve

In the mean time I'll work on adding this to the tool natively. Even if it is installed from outside BusinessOps.

Cheers, Marc

marclelijveld commented 2 years ago

Fixed in latest release #3c8a0cf