Skip to content

chore: add tsg link in error messages#43597

Merged
jorgerangel-msft merged 4 commits intoAzure:mainfrom
jorgerangel-msft:appconfig-tsg
May 10, 2024
Merged

chore: add tsg link in error messages#43597
jorgerangel-msft merged 4 commits intoAzure:mainfrom
jorgerangel-msft:appconfig-tsg

Conversation

@jorgerangel-msft
Copy link
Copy Markdown
Member

@jorgerangel-msft jorgerangel-msft commented Apr 23, 2024

This PR enhances the error handling done by the client by appending a troubleshooting message that contains the app config TSG link to error messages returned by operations within the client.

@github-actions github-actions Bot added the App Configuration Azure.ApplicationModel.Configuration label Apr 23, 2024
@azure-sdk
Copy link
Copy Markdown
Collaborator

API change check

API changes are not detected in this pull request.

@jorgerangel-msft jorgerangel-msft marked this pull request as ready for review April 23, 2024 21:40
Comment thread sdk/appconfiguration/Azure.Data.AppConfiguration/src/ConfigurationClient.cs Outdated
Comment thread sdk/appconfiguration/Azure.Data.AppConfiguration/src/ConfigurationClient.cs Outdated
Comment thread sdk/appconfiguration/Azure.Data.AppConfiguration/tests/ConfigurationLiveTests.cs Outdated
Comment thread sdk/appconfiguration/Azure.Data.AppConfiguration/src/ConfigurationClient.cs Outdated
Copy link
Copy Markdown
Contributor

@kinelski kinelski left a comment

Choose a reason for hiding this comment

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

Changes LGTM!

Copy link
Copy Markdown
Member

@jsquire jsquire left a comment

Choose a reason for hiding this comment

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

LGTM

@jorgerangel-msft jorgerangel-msft merged commit c4418cd into Azure:main May 10, 2024
@jorgerangel-msft jorgerangel-msft deleted the appconfig-tsg branch May 10, 2024 21:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

App Configuration Azure.ApplicationModel.Configuration

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[AppConfig] Add aka.ms links from SDK exception messages to anchors in the TSG

5 participants