Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

PnP authentication update required for installation/M365lpConfiguration.ps1 #854

Open
smiddles opened this issue Oct 18, 2024 · 6 comments

Comments

@smiddles
Copy link

PROVISIONING NOTE: If you are having issues with the SharePoint look book while trying to provision M365 learning pathways please do not post that issue to this repo as we cannot help you here. Please post all questions and issues for the SharePoint look book to SharePoint/sp-provisioning-service GitHub Issues List.

WEB PART NOT LOADING: If you are getting the error: "Microsoft 365 learning pathways has a configuration issue. Ask your administrator for assistance". Please review the Diagnosing Pre-Check guidance before submitting your bug.

Describe the bug

Unable to authenticate to a SharePoint Online tenant. September 2024 updates to authentication of PnP Powershell and the requirement for a Client ID to be used is not currently configured in the installation/M365lpConfiguration.ps1 script.

To Reproduce

Execute script, enter valid admin credentials (to SharePoint Admin).

Expected behavior

Continue script creation of pages to nominated Learning Pathways communication site

Screenshots

If applicable, add screenshots to help explain your problem.

Learning Pathways version number

Please share the version number of learning pathways in your tenant app catalog.

Additional context

Add any other context about the problem here.

@dcashpeterson
Copy link
Contributor

@smiddles Thanks for bringing this to our attention. What would your expectation be for the edit to the script? Would you expect a parameter for client ID or something else?

@smiddles
Copy link
Author

smiddles commented Oct 24, 2024 via email

@dcashpeterson dcashpeterson self-assigned this Oct 28, 2024
@badwolf904
Copy link

image

Does the new ClientID variable fix this issue? Still getting this error.

@dcashpeterson
Copy link
Contributor

@badwolf904 If you have a copy of the script downloaded you can add the -ClientId property and value to the script with your organizations app registration ID. The URL in the error you recieved tells you how to do that if it is not already done. I have not had a chance to make the Client Id a property on the script yet.

@badwolf904
Copy link

Do you know specifically where it needs to go in the script? I tried unsuccessfully. If not I'll wait for your edit :)

@dcashpeterson
Copy link
Contributor

@badwolf904 you add it anywhere you see PnP.PowerShell\Connect-PnPOnline however, if you are not comfortable with PowerShell I would not make the change yourself. This is a moderately complex script and if you change the wrong thing it might not work correctly.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants