search

pnp / custom-learning-office-365

Microsoft Learning Pathways end user learning solution for Microsoft 365 customers.
MIT License
228 stars 209 forks source link

Update manual configuration to use latest PnP PowerShell #706

Closed kevmcdonk closed 11 months ago

kevmcdonk commented 1 year ago

The current manual configuration PowerShell requires the legacy PnP PowerShell which many people no longer have installed.

I would like to see a new version of this that uses the latest PnP PowerShell - 1.12.0.

This would be needed at some point and could either replace the existing Configuration file or provide an alternative version of it (possibly the better option for supporting both scenarios).

I am happy to work on this (seeing as I started doing it already on my machine) but wanted to make sure that it fits with the longer term goals.

dcashpeterson commented 1 year ago

@kevmcdonk reach out to @ToddKlindt he was working on updating the script to support both in one script a while back. The challenge here is that the updated version will require the app reg to run the PowerShell which many folks don't have the ability to do. This will end up causing troubleshooting issues on our end. If you are offering we will gladly accept the update as long as you can make one script to support both depending on what is installed. We should also update the documentation after the script is updated. Thanks.

kevmcdonk commented 1 year ago

So would a single script that supported both be preferable to two script that allows the user to pick?

(And thanks for the quick response!)

dcashpeterson commented 1 year ago

Yes, a single script would be better.

kevmcdonk commented 1 year ago

Added PR #709 for this one

dcashpeterson commented 11 months ago

Merged the PR and closing this. Thanks for the assistance

github-actions[bot] commented 10 months ago

This issue is locked for inactivity or age. If you have a related issue please open a new issue and reference this one. Closed issues are not tracked.