Skip to content

Conversation

katriendg
Copy link
Contributor

This PR enhances the troubleshooting documentation by adding guidance for resolving authentication issues in Dev Container and WSL environments, expanding on the existing multi-tenant authentication troubleshooting section.

Changes Made

  • Enhanced Multi-Tenant Authentication Section: Added detailed configuration examples for both public feed and source installations with tenant ID support
  • New Dev Container/WSL Authentication Section: Added comprehensive troubleshooting guidance for authentication issues specific to Dev Container and WSL environments

GitHub issue number

#398 - Documentation: clarify troubleshooting document for multi-tenant setup and add section on WSL/User global settings

Associated Risks

Documentation update only

PR Checklist

  • I have read the contribution guidelines
  • I have read the code of conduct guidelines
  • Title of the pull request is clear and informative.
  • 👌 Code hygiene
  • 🔭 Telemetry added, updated, or N/A
  • 📄 Documentation added, updated, or N/A
  • 🛡️ Automated tests added, or N/A

🧪 How did you test it?

The described approach is the one I had to take to make the ADO MCP work in WSL and Dev Container (while keeping the settings at global User level), and could be useful to help others troubleshooting a root cause that might be confusing to discover at first.

… issues and provide detailed configuration examples for MCP server setup in Dev Containers and WSL environments.
@katriendg katriendg requested a review from a team as a code owner August 6, 2025 06:59
@codecov-commenter
Copy link

codecov-commenter commented Aug 6, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
⚠️ Please upload report for BASE (main@6cf8a2a). Learn more about missing BASE report.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #401   +/-   ##
=======================================
  Coverage        ?   77.26%           
=======================================
  Files           ?       13           
  Lines           ?      805           
  Branches        ?      132           
=======================================
  Hits            ?      622           
  Misses          ?      171           
  Partials        ?       12           
Flag Coverage Δ
unittests 77.26% <ø> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@danhellem
Copy link
Contributor

@katriendg please fix formatting issues by npm run format

@danhellem danhellem added Documentation 📙 improvements or additions to documentation Needs Review 👓 needs review by the product team labels Aug 6, 2025
@danhellem danhellem self-assigned this Aug 6, 2025
@katriendg
Copy link
Contributor Author

Sorry I missed that note in the guidance. Done.

@danhellem danhellem merged commit bd99d30 into microsoft:main Aug 6, 2025
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Documentation 📙 improvements or additions to documentation Needs Review 👓 needs review by the product team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Documentation: clarify troubleshooting document for multi-tenant setup and add section on WSL/User global settings

3 participants