Skip to content

Conversation

vanpeltgregory
Copy link
Contributor

Update SDK Samples links and remove references to cognitive services

Update SDK Samples links and remove references to cognitive services
Copy link

Learn Build status updates of commit ab5f888:

💡 Validation status: suggestions

File Status Preview URL Details
bing-docs/bing-web-search/quickstarts/sdk/web-search-client-library-csharp.md 💡Suggestion View Details
bing-docs/bing-web-search/quickstarts/sdk/web-search-client-library-java.md 💡Suggestion View Details
bing-docs/bing-web-search/quickstarts/sdk/web-search-client-library-javascript.md 💡Suggestion View Details
bing-docs/bing-web-search/quickstarts/sdk/web-search-client-library-python.md 💡Suggestion View Details
bing-docs/bing-web-search/samples.md 💡Suggestion View Details

bing-docs/bing-web-search/quickstarts/sdk/web-search-client-library-csharp.md

  • Line 12, Column 12: [Suggestion: ms-author-invalid - See documentation] Invalid value for 'ms.author', 'scottwhi' is not a valid Microsoft alias.

bing-docs/bing-web-search/quickstarts/sdk/web-search-client-library-java.md

  • Line 12, Column 12: [Suggestion: ms-author-invalid - See documentation] Invalid value for 'ms.author', 'scottwhi' is not a valid Microsoft alias.

bing-docs/bing-web-search/quickstarts/sdk/web-search-client-library-javascript.md

  • Line 12, Column 12: [Suggestion: ms-author-invalid - See documentation] Invalid value for 'ms.author', 'scottwhi' is not a valid Microsoft alias.

bing-docs/bing-web-search/quickstarts/sdk/web-search-client-library-python.md

  • Line 12, Column 12: [Suggestion: ms-author-invalid - See documentation] Invalid value for 'ms.author', 'scottwhi' is not a valid Microsoft alias.

bing-docs/bing-web-search/samples.md

  • Line 12, Column 12: [Suggestion: ms-author-invalid - See documentation] Invalid value for 'ms.author', 'scottwhi' is not a valid Microsoft alias.

For more details, please refer to the build report.

Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.

For any questions, please:

@GhiathAjam
Copy link
Contributor

Please change ms.author to your Microsoft alias.

@vanpeltgregory
Copy link
Contributor Author

@microsoft-github-policy-service agree company="Microsoft"

Update SDK Samples/Quick Start links and remove references to cognitive services
Copy link

Learn Build status updates of commit 17d3cd5:

💡 Validation status: suggestions

File Status Preview URL Details
bing-docs/bing-web-search/quickstarts/sdk/web-search-client-library-javascript.md 💡Suggestion View Details
bing-docs/bing-web-search/quickstarts/sdk/web-search-client-library-csharp.md ✅Succeeded View
bing-docs/bing-web-search/quickstarts/sdk/web-search-client-library-java.md ✅Succeeded View
bing-docs/bing-web-search/quickstarts/sdk/web-search-client-library-python.md ✅Succeeded View
bing-docs/bing-web-search/samples.md ✅Succeeded View

bing-docs/bing-web-search/quickstarts/sdk/web-search-client-library-javascript.md

  • Line 12, Column 12: [Suggestion: ms-author-invalid - See documentation] Invalid value for 'ms.author', 'scottwhi' is not a valid Microsoft alias.

For more details, please refer to the build report.

Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.

For any questions, please:

@@ -317,4 +317,4 @@ When you're done with this project, make sure to remove your subscription key fr
## Next steps

> [!div class="nextstepaction"]
Copy link
Contributor

Choose a reason for hiding this comment

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

Is removing this ! intentional? what's the motive?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Need clarification on that feedback; I don't recall removing it.

Copy link
Contributor

Choose a reason for hiding this comment

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

Must've been removed by mistake then, please double check and add it back if necessary.

@GhiathAjam
Copy link
Contributor

please modify ms.author for the file mentioned.

ms author changed to valid value for this commit in one flagged file from validation
Copy link

Learn Build status updates of commit e7b48ff:

💡 Validation status: suggestions

File Status Preview URL Details
bing-docs/bing-web-search/quickstarts/sdk/web-search-client-library-javascript.md 💡Suggestion View Details
bing-docs/bing-web-search/quickstarts/sdk/web-search-client-library-csharp.md ✅Succeeded View
bing-docs/bing-web-search/quickstarts/sdk/web-search-client-library-java.md ✅Succeeded View
bing-docs/bing-web-search/quickstarts/sdk/web-search-client-library-python.md ✅Succeeded View
bing-docs/bing-web-search/samples.md ✅Succeeded View

bing-docs/bing-web-search/quickstarts/sdk/web-search-client-library-javascript.md

  • Line 12, Column 12: [Suggestion: ms-author-invalid - See documentation] Invalid value for 'ms.author', 'v-grvanp' is not a valid Microsoft alias.

For more details, please refer to the build report.

Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.

For any questions, please:

@@ -9,7 +9,7 @@ ms.service: bing-search-services
ms.subservice: bing-web-search
ms.topic: quickstart
ms.date: 07/15/2020
ms.author: scottwhi
ms.author: v-grvanp
Copy link
Contributor

Choose a reason for hiding this comment

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

looks different than the other files v-grvanp vs v-grvanpelt, typo?

Fixed the ms.author typo on line 12
Copy link

Learn Build status updates of commit 71e2e91:

✅ Validation status: passed

File Status Preview URL Details
bing-docs/bing-web-search/quickstarts/sdk/web-search-client-library-csharp.md ✅Succeeded View
bing-docs/bing-web-search/quickstarts/sdk/web-search-client-library-java.md ✅Succeeded View
bing-docs/bing-web-search/quickstarts/sdk/web-search-client-library-javascript.md ✅Succeeded View
bing-docs/bing-web-search/quickstarts/sdk/web-search-client-library-python.md ✅Succeeded View
bing-docs/bing-web-search/samples.md ✅Succeeded View

For more details, please refer to the build report.

For any questions, please:

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