Skip to content

Add Token Caching Identity documentation.#34389

Merged
g2vinay merged 6 commits into
Azure:mainfrom
g2vinay:add-identity-credential-caching-doc
May 8, 2023
Merged

Add Token Caching Identity documentation.#34389
g2vinay merged 6 commits into
Azure:mainfrom
g2vinay:add-identity-credential-caching-doc

Conversation

@g2vinay

@g2vinay g2vinay commented Apr 10, 2023

Copy link
Copy Markdown
Member

Fixes #31474

Adds Token Caching doc to the azure-identity module.
Adds a Token Caching section in the README and a hyperlink to Token caching doc.

@ghost ghost added the Azure.Identity label Apr 10, 2023
@g2vinay g2vinay requested a review from scottaddie April 10, 2023 16:27
@azure-sdk

Copy link
Copy Markdown
Collaborator

API change check

API changes are not detected in this pull request.

Comment thread sdk/identity/azure-identity/README.md Outdated
Comment thread sdk/identity/azure-identity/README.md Outdated
Comment thread sdk/identity/azure-identity/TOKEN_CACHING.md Outdated
Comment thread sdk/identity/azure-identity/TOKEN_CACHING.md Outdated
Comment thread sdk/identity/azure-identity/README.md Outdated
Comment thread sdk/identity/azure-identity/README.md Outdated
Comment thread sdk/identity/azure-identity/TOKEN_CACHING.md Outdated
Comment thread sdk/identity/azure-identity/TOKEN_CACHING.md
Comment thread sdk/identity/azure-identity/TOKEN_CACHING.md Outdated
Comment thread sdk/identity/azure-identity/TOKEN_CACHING.md Outdated
Comment thread sdk/identity/azure-identity/TOKEN_CACHING.md
Comment thread sdk/identity/azure-identity/TOKEN_CACHING.md Outdated
Comment thread sdk/identity/azure-identity/TOKEN_CACHING.md
Comment thread sdk/identity/azure-identity/TOKEN_CACHING.md Outdated
Comment thread sdk/identity/azure-identity/TOKEN_CACHING.md
@g2vinay g2vinay requested review from schaabs and scottaddie April 21, 2023 21:38
Comment thread sdk/identity/azure-identity/TOKEN_CACHING.md Outdated
Comment thread sdk/identity/azure-identity/TOKEN_CACHING.md Outdated
Comment thread sdk/identity/azure-identity/README.md Outdated
Comment thread sdk/identity/azure-identity/TOKEN_CACHING.md Outdated
Comment thread sdk/identity/azure-identity/TOKEN_CACHING.md Outdated
Comment thread sdk/identity/azure-identity/TOKEN_CACHING.md Outdated
Comment thread sdk/identity/azure-identity/TOKEN_CACHING.md
Comment thread sdk/identity/azure-identity/TOKEN_CACHING.md Outdated
Comment thread sdk/identity/azure-identity/TOKEN_CACHING.md Outdated
Comment thread sdk/identity/azure-identity/README.md Outdated
Comment thread sdk/identity/azure-identity/README.md Outdated
Comment thread sdk/identity/azure-identity/README.md Outdated

@scottaddie scottaddie left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

A few more suggestions to consider, and then this PR looks good to me.

Comment thread sdk/identity/azure-identity/TOKEN_CACHING.md Outdated
Comment thread sdk/identity/azure-identity/README.md Outdated
Comment thread sdk/identity/azure-identity/README.md Outdated
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.

Create documentation for Identity credential cache

5 participants