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

QuickStart Readme Files and This Updates The QuickStart so they match Doctsite. #1588

Open
wants to merge 32 commits into
base: master
Choose a base branch
from

Conversation

KilloranFiander
Copy link
Contributor

Purpose

Updated Pivot point so that the link takes you to the C++ QuickStart.

Pull Request Type

What kind of change does this Pull Request introduce?

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[X ] Documentation content changes
[ ] Other... Please describe:

@KilloranFiander KilloranFiander marked this pull request as ready for review July 25, 2022 15:48
Copy link
Contributor Author

@KilloranFiander KilloranFiander left a comment

Choose a reason for hiding this comment

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

Additional information added to pull.

* * A subscription key and region for the Speech service.
* Azure subscription - [Create one for free](https://azure.microsoft.com/en-us/free/cognitive-services/)
* [Create a Speech resource](https://ms.portal.azure.com/#create/Microsoft.CognitiveServicesSpeechServices) in the Azure portal.
* Get the resource key and region. After your Speech resource is deployed, select Go to resource to view and manage keys. or more information about Cognitive Services resources, see [Get the keys for your resource](https://docs.microsoft.com/en-us/azure/cognitive-services/cognitive-services-apis-create-account?tabs=multiservice%2Cwindows#get-the-keys-for-your-resource)
Copy link
Member

Choose a reason for hiding this comment

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

"Go to resource" is bold on the docs.microsoft.com view for this and I think it's important to visually distinguish it.

Could you please update all instances of this with enclosing double asterisks? That'll make it bold, e.g.:
**this text**
becomes this text

@@ -36,5 +39,5 @@ To debug the app and then run it, press F5 or use **Debug** \> **Start Debugging

## References

* [Quickstart article on the SDK documentation site](https://docs.microsoft.com/azure/cognitive-services/speech-service/quickstart-text-to-speech-cpp-windows)
* [Quickstart article on the SDK documentation site](https://docs.microsoft.com/en-us/azure/cognitive-services/speech-service/get-started-text-to-speech?tabs=terminal&pivots=programming-language-cpp)
Copy link
Member

Choose a reason for hiding this comment

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

Please remove the en-us/ part from all the links; the docs site will automatically add in the appropriate region where appropriate.

@KilloranFiander KilloranFiander changed the title Update The C++ QuickStart Link Pivot to Language Update the QuickStart Link Pivot to Multiple Languages Jul 27, 2022
Copy link
Contributor Author

@KilloranFiander KilloranFiander left a comment

Choose a reason for hiding this comment

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

Completed Updates to the links and documentation.

@KilloranFiander
Copy link
Contributor Author

Pull has been completed.

@KilloranFiander KilloranFiander changed the title Update the QuickStart Link Pivot to Multiple Languages QuickStart Readme Files and This Updates The QuickStart so they match Doctsite. Aug 18, 2022
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.

2 participants