Skip to content

.Net: docs: fix dead links after planner removal and MEVD code migration - #14715

Merged
westey (westey-m) merged 1 commit into
microsoft:mainfrom
leom1919:docs-fix-migrated-sample-links
Nov 3, 2026
Merged

westey (westey-m) merged 1 commit into
microsoft:mainfrom
leom1919:docs-fix-migrated-sample-links

Conversation

@leom1919

@leom1919 leom1919 commented Nov 3, 2026

Copy link
Copy Markdown
Contributor

Motivation and Context

A full link-integrity scan of the repository's README files found 13 dead internal links in 4 files. The underlying sample files were deleted or moved by recent refactors (planner removal #12399, MEVD Preview 2 #11028, MsGraph V5 update #12962, and the Gemini thinking-budget sample rename in #13396), but the README entries still point at the old paths.

Description

Verification

All 5 new URLs were checked and return HTTP 200. All 13 removed/updated links were verified to return HTTP 404 against main before this change.

Docs-only change: no code is touched, so builds and unit tests are unaffected.

Fix 13 dead links across 4 README files:

- dotnet/samples/Concepts/README.md: update Google_GeminiChatCompletionWithThinkingBudget to its renamed file Google_GeminiChatCompletionWithThinking (renamed in microsoft#13396); remove the VectorStore_DataIngestion_CustomMapper entry (sample deleted in microsoft#11028); remove the entire Planners section (samples deleted in microsoft#12399); remove MsGraph_ContactsPlugin/DrivePlugin/TasksPlugin entries (no longer present after MsGraph V5 update in microsoft#12962); remove WithFunctionCallingStepwisePlanner entry (deleted with the planners in microsoft#12399)
- dotnet/samples/Demos/OnnxSimpleRAG/README.md: point Vector Store to Microsoft.Extensions.VectorData.Abstractions (moved to dotnet/extensions) and InMemoryVectorStore to CommunityToolkit/AI
- python/semantic_kernel/connectors/memory_stores/redis/README.md: point .NET README link to dotnet/src/VectorData/Redis/README.md
- python/samples/concepts/README.md: point Search section link to python/semantic_kernel/connectors/search.py
Copilot AI balanced review requested due to automatic review settings November 3, 2026 06:10
@leom1919
leom1919 requested review from a team as code owners November 3, 2026 06:10
@leom1919
leom1919 deployed to github-app-auth November 3, 2026 06:10 — with GitHub Actions Active
@semantic-kernel-automation semantic-kernel-automation Bot added .NET Issue or Pull requests regarding .NET code python Pull requests for the Python Semantic Kernel documentation labels Nov 3, 2026
@github-actions github-actions Bot changed the title docs: fix dead links after planner removal and MEVD code migration Python: docs: fix dead links after planner removal and MEVD code migration Nov 3, 2026
@github-actions github-actions Bot changed the title Python: docs: fix dead links after planner removal and MEVD code migration .Net: docs: fix dead links after planner removal and MEVD code migration Nov 3, 2026

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.

🟢 Approval recommended

No active review issues remain for the documentation link updates.

No open issues 🎉

What changed in this PR

This documentation-only PR repairs README links after planner removal, VectorData migration, and sample refactors. No runtime code changes are included.

Changes:

  • Removes links to deleted planner, ingestion, and Microsoft Graph samples.
  • Updates the Gemini sample link to its renamed file.
  • Redirects VectorData, Redis, and Python search links to their current locations.
File Description
python/​semantic_kernel/​connectors/​memory_stores/​redis/​README.md Points to the current .NET Redis README.
python/​samples/​concepts/​README.md Updates the search link to connectors/search.py.
dotnet/​samples/​Demos/​OnnxSimpleRAG/​README.md Redirects VectorData and in-memory store links to their upstream repositories.
dotnet/​samples/​Concepts/​README.md Updates the Gemini sample link and removes deleted sample entries.

Review effort: Balanced

@westey-m
westey (westey-m) added this pull request to the merge queue Nov 3, 2026
Merged via the queue into microsoft:main with commit 9137b35 Nov 3, 2026
39 checks passed

This branch was successfully deployed

1 active deployment
github-app-auth — 19881da3 Deployed Nov 3, 2026 by leom1919 via add_label #29568
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation .NET Issue or Pull requests regarding .NET code python Pull requests for the Python Semantic Kernel

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants