Skip to content

fix: Update Azure App Configuration documentation to clarify connecti… - #108

Merged
David Pine (IEvangelist) merged 4 commits into
mainfrom
fix-106
Dec 5, 2025
Merged

fix: Update Azure App Configuration documentation to clarify connecti…#108
David Pine (IEvangelist) merged 4 commits into
mainfrom
fix-106

Conversation

@IEvangelist

@IEvangelist David Pine (IEvangelist) commented Dec 2, 2025

Copy link
Copy Markdown
Member

…on methods and emulator usage - fixes #106

/cc Zhiyuan Liang (@zhiyuanliang-ms)

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the Azure App Configuration documentation to clarify connection methods and emulator usage, addressing issue #106. The documentation has been restructured to better explain how developers can connect to Azure App Configuration stores using different approaches.

Key changes:

  • Clarified connection methods, replacing the AsExisting API documentation with AddConnectionString approach
  • Added comprehensive documentation for using the Azure App Configuration emulator, including container configuration options
  • Updated client integration section to focus on the configuration provider approach rather than the ConfigurationClient API

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

You can also share your feedback on Copilot code review for a chance to win a $100 gift card. Take the survey.

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@zhiyuanliang-ms

Copy link
Copy Markdown
Member

Hey, David Pine (@IEvangelist)
Thank you for this PR.

@IEvangelist

Copy link
Copy Markdown
Member Author

Hey Copilot, please address new review feedback.

Copilot AI commented Dec 5, 2025

Copy link
Copy Markdown
Contributor

David Pine (@IEvangelist) I've opened a new pull request, #119, to work on those changes. Once the pull request is ready, I'll request review from you.

* Initial plan

* Address zhiyuanliang-ms review feedback: add configureOptions section, clarify data persistence

Co-authored-by: IEvangelist <7679720+IEvangelist@users.noreply.github.com>

* Improve readability per code review feedback

Co-authored-by: IEvangelist <7679720+IEvangelist@users.noreply.github.com>

* Fix formatting and update import statements in MDX

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: IEvangelist <7679720+IEvangelist@users.noreply.github.com>
Co-authored-by: David Pine <david.pine@microsoft.com>
@IEvangelist
David Pine (IEvangelist) merged commit 38f6f9e into main Dec 5, 2025
7 checks passed
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.

Azure App Configuration documentation is wrong

5 participants