Skip to content

Conversation

msbemba
Copy link
Contributor

@msbemba msbemba commented Jun 23, 2023

removed [XML] from example 5 for the cmdlet to work

Fixes #3042

removed [XML] from example 5 for the cmdlet to work

Fixes MicrosoftDocs#3042
@msbemba
Copy link
Contributor Author

msbemba commented Jun 23, 2023

@JohanFreelancer9 Please copy review

@msbemba msbemba mentioned this pull request Jun 23, 2023
@learn-build-service-prod
Copy link
Contributor

Learn Build status updates of commit 1941c78:

✅ Validation status: passed

File Status Preview URL Details
docset/winserver2022-ps/vpnclient/Add-VpnConnection.md ✅Succeeded View (WindowsServer2022-ps)

For more details, please refer to the build report.

For any questions, please:

@learn-build-service-prod
Copy link
Contributor

Learn Build status updates of commit 51fc09f:

✅ Validation status: passed

File Status Preview URL Details
docset/winserver2012-ps/vpnclient/Add-VpnConnection.md ✅Succeeded View (winserver2012-ps)
docset/winserver2022-ps/vpnclient/Add-VpnConnection.md ✅Succeeded View (WindowsServer2022-ps)

For more details, please refer to the build report.

For any questions, please:

@scanum scanum assigned ghost , get-itips and scanum Jul 3, 2023
### Example 5: Add a VPN connection that uses already generated EAP XML configuration
```powershell
PS C:\> $EAPXml = [xml]Get-Content -Path C:\eap-config.xml
PS C:\> $EAPXml = Get-Content -Path C:\eap-config.xml
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do you feel there is a need to define $EAPXml as [xml] instead (PS C:\> [xml]$EAPXml)?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I thought so too. However, there is a chance they would copy-paste that as well and report errors like what we have seen now.

@tiburd tiburd merged commit 292c879 into MicrosoftDocs:main Jul 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Example 5 not valid

5 participants