-
Notifications
You must be signed in to change notification settings - Fork 218
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
Comments
@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? |
Thanks Derek. Adding a variable declaration at the top for the ClientID, which is then applied in the instances of Connect-PnPOnline if declared?
…________________________________
From: Derek Cash-Peterson ***@***.***>
Sent: Tuesday, 22 October 2024 2:28 AM
To: pnp/custom-learning-office-365 ***@***.***>
Cc: Steve Middleton ***@***.***>; Mention ***@***.***>
Subject: Re: [pnp/custom-learning-office-365] PnP authentication update required for installation/M365lpConfiguration.ps1 (Issue #854)
@smiddles<https://github.com/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?
—
Reply to this email directly, view it on GitHub<#854 (comment)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/BMG22XTIGUT6OS5RHA2ZFR3Z4UTSDAVCNFSM6AAAAABQE6H7A6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIMRXGE2TMNRSGA>.
You are receiving this because you were mentioned.Message ID: ***@***.***>
|
@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. |
Do you know specifically where it needs to go in the script? I tried unsuccessfully. If not I'll wait for your edit :) |
@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. |
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
Learning Pathways version number
Additional context
The text was updated successfully, but these errors were encountered: