I believe there is a line break missing in the PowerShell code at https://docs.microsoft.com/en-us/surface-hub/surface-hub-2s-account#connect-to-exchange-online-powershell
⚠Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.
@officedocsbot assign @jvsam
@hammar : Can you confirm that the cmdlet should look like my suggestion below?
$365Session = New-PSSession -ConfigurationName Microsoft.Exchange -ConnectionUri https://ps.outlook.com/powershell -Credential (Get-Credential) -Authentication Basic –AllowRedirection
$ImportResults = Import-PSSession $365Session
As you have read this page more intently than me, have you noticed any of the other cmdlets in need of added line breaks?
edit: I could be mistaken, but it looks like line 4 below either needs to be commented out or separated as the title of a subsequent code block:
Import-Module LyncOnlineConnector
$SfBSession = New-CsOnlineSession -Credential (Get-Credential)
Import-PSSession $SfBSession -AllowClobber
Enable the Skype for Business meeting room
Enable-CsMeetingRoom -Identity [email protected] -RegistrarPool(Get-CsTenant).Registrarpool -SipAddressType EmailAddress
Import-Module LyncOnlineConnector
$SfBSession = New-CsOnlineSession -Credential (Get-Credential)
Import-PSSession $SfBSession -AllowClobber
Enable-CsMeetingRoom -Identity [email protected] -RegistrarPool(Get-CsTenant).Registrarpool -SipAddressType EmailAddress
@illfated correct on both counts; the line break in your first suggestion is where I think it should be, and the fourth line on the subsequent example should indeed be commented out (that makes more sense than introducing a new title, as line 5 is a direct continuation of lines 1-3).
Thanks a lot for the useful feedback. Based on what you have told me, I guess it could look something like this instead:
Import-Module LyncOnlineConnector
$SfBSession = New-CsOnlineSession -Credential (Get-Credential)
Import-PSSession $SfBSession -AllowClobber
# Enable the Skype for Business meeting room
Enable-CsMeetingRoom -Identity [email protected] -RegistrarPool(Get-CsTenant).Registrarpool -SipAddressType EmailAddress
(I added a blank line for readability, but you are welcome to tell me if that kind of detail is less useful in this case.
Looks good to me.
Very well, looks good to me too. I will get back to this later tonight.
For now, I need to get some delayed support tasks off my schedule.
That's great @illfated. Were you able to test the commands? Looking forward to your PR.
Thanks as well @hammar for this feedback, much appreciated. We always strive to give users the best Windows 10 experience by ensuring that contents of the Microsoft Docs are useful, accurate and up-to-date.
Thank you for asking. Unfortunately, I don't have a test scenario where these commands will be of any practical use.
For now, I can only use logic and reason to see what needs to be done.
If anyone else would like to confirm that the suggested changes work as expected, I am all for it.
There you go. PR #5471 has been opened.
To see the full differences, view the Github document pages:
😲 ... and there it is merged already. @hammar : Thanks for the assistance,
I must admit, I have rarely seen such quick response as this evening/night.