Skip to content

Update text for Set-CsOnlineEnhancedEmergencyServiceDisclaimer #8373

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

Merged
merged 16 commits into from
Sep 28, 2021

Conversation

JensTrier
Copy link
Contributor

Updated the description and synopsis to make it clearer what this cmdlet does

@opbld31
Copy link

opbld31 commented Sep 23, 2021

Docs Build status updates of commit 50883ee:

✅ Validation status: passed

File Status Preview URL Details
skype/skype-ps/skype/Set-CsOnlineEnhancedEmergencyServiceDisclaimer.md ✅Succeeded View (skype-ps)

For more details, please refer to the build report.

Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.

For any questions, please:

…aimer.md

Co-authored-by: Andres Mariano Gorzelany <36666927+get-itips@users.noreply.github.com>
Copy link
Contributor

@dariomws dariomws left a comment

Choose a reason for hiding this comment

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

Some minor suggestions.

Not sure about add Teams to the applicable section. All the other cmdlets only include Skype for Business Online.

JensTrier and others added 3 commits September 23, 2021 16:11
…aimer.md

Co-authored-by: Andres Mariano Gorzelany <36666927+get-itips@users.noreply.github.com>
…aimer.md

Co-authored-by: Dario Woitasen <33589238+dariomws@users.noreply.github.com>
…aimer.md

Co-authored-by: Andres Mariano Gorzelany <36666927+get-itips@users.noreply.github.com>
@opbld30
Copy link

opbld30 commented Sep 23, 2021

Docs Build status updates of commit 0c3d62d:

✅ Validation status: passed

File Status Preview URL Details
skype/skype-ps/skype/Set-CsOnlineEnhancedEmergencyServiceDisclaimer.md ✅Succeeded View (skype-ps)

For more details, please refer to the build report.

Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.

For any questions, please:

@JensTrier
Copy link
Contributor Author

Some minor suggestions.

Not sure about add Teams to the applicable section. All the other cmdlets only include Skype for Business Online.

I can remove Microsoft Teams from Applicable section, just let me know

@opbld33
Copy link

opbld33 commented Sep 23, 2021

Docs Build status updates of commit 98f54f8:

✅ Validation status: passed

File Status Preview URL Details
skype/skype-ps/skype/Set-CsOnlineEnhancedEmergencyServiceDisclaimer.md ✅Succeeded View (skype-ps)

For more details, please refer to the build report.

Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.

For any questions, please:

Copy link
Contributor

@get-itips get-itips left a comment

Choose a reason for hiding this comment

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

Please correct the other Applicable occurrences, thanks

@opbld32
Copy link

opbld32 commented Sep 23, 2021

Docs Build status updates of commit 7e46617:

✅ Validation status: passed

File Status Preview URL Details
skype/skype-ps/skype/Set-CsOnlineEnhancedEmergencyServiceDisclaimer.md ✅Succeeded View (skype-ps)

For more details, please refer to the build report.

Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.

For any questions, please:

@dariomws
Copy link
Contributor

Some minor suggestions.
Not sure about add Teams to the applicable section. All the other cmdlets only include Skype for Business Online.

I can remove Microsoft Teams from Applicable section, just let me know

Let's add it, there are a few more cmdlets with the same information and I think it's a good idea.

@JensTrier
Copy link
Contributor Author

Some minor suggestions.
Not sure about add Teams to the applicable section. All the other cmdlets only include Skype for Business Online.

I can remove Microsoft Teams from Applicable section, just let me know

Let's add it, there are a few more cmdlets with the same information and I think it's a good idea.

I added it.

@opbld32
Copy link

opbld32 commented Sep 24, 2021

Docs Build status updates of commit bb012f9:

✅ Validation status: passed

File Status Preview URL Details
skype/skype-ps/skype/Set-CsOnlineEnhancedEmergencyServiceDisclaimer.md ✅Succeeded View (skype-ps)

For more details, please refer to the build report.

Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.

For any questions, please:

@JensTrier
Copy link
Contributor Author

@get-itips do you need more changes?

@yogkumgit
Copy link

@tiburd Please merge this PR. Thanks

@opbld32
Copy link

opbld32 commented Sep 28, 2021

Docs Build status updates of commit a466428:

✅ Validation status: passed

File Status Preview URL Details
skype/skype-ps/skype/Set-CsOnlineEnhancedEmergencyServiceDisclaimer.md ✅Succeeded View (skype-ps)

For more details, please refer to the build report.

Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.

For any questions, please:

@tiburd tiburd merged commit 38e059c into MicrosoftDocs:master Sep 28, 2021
@JensTrier JensTrier deleted the emergencydisclaimer2 branch September 29, 2021 12:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

10 participants