Update sample readme slug #18009
Update sample readme slug #18009
Conversation
| products: | ||
| - azure | ||
| - azure-identity | ||
| - azure-sdks |
There was a problem hiding this comment.
JS uses azure-active-directory here. Would it make sense to use the same?
There was a problem hiding this comment.
Would that cause these samples to show up in Azure AD documentation?
There was a problem hiding this comment.
Answered my own question: it does, in the samples browser at least. At first glance I'm thinking our samples shouldn't be grouped with Azure Active Directory samples. This library uses Azure AD but is narrowly focused on authenticating Azure SDK clients. Our samples are poor demonstrations of Azure AD features and in general just noise for people not using the SDK.
There was a problem hiding this comment.
Where would someone expect to find the identity SDK samples in the microsoft docs?
There was a problem hiding this comment.
For example, in form recognizer, our MS docs menu has a drop down for "Samples" where our samples readme gets published: https://docs.microsoft.com/en-us/azure/cognitive-services/form-recognizer/overview?tabs=v2-1
There was a problem hiding this comment.
Authentication is covered in the SDK conceptual docs. As for the sample browser, the taxonomy is ordered by product, so I think azure-identity samples belong under azure (though I think that's too broad a category to be really useful) and azure-sdks.
| - python | ||
| products: | ||
| - azure | ||
| - azure-identity |
There was a problem hiding this comment.
Is an identity tag necessary for these to show up with Identity samples?
There was a problem hiding this comment.
I believe so but question whether we want these samples to show up there. Today there's no "there" there; the sample browser has nothing for "identity". The sample browser covers products outside Azure, so not knowing the intended scope of "identity", I'm reluctant to make our samples the only ones in that category.
3 participants
Closes #17994