Skip to content

.NET: Pin the documentation pages of the RAG Step02 sample to a commit - #9648

Open
Federico Laurianti (Laurianti) wants to merge 1 commit into
microsoft:mainfrom
Laurianti:fix-step02-doc-links
Open

Federico Laurianti (Laurianti) wants to merge 1 commit into
microsoft:mainfrom
Laurianti:fix-step02-doc-links

Conversation

@Laurianti

Copy link
Copy Markdown
Contributor

Motivation & Context

AgentWithRAG_Step02_CustomVectorStoreRAG downloads two pages from the main branch of MicrosoftDocs/semantic-kernel-docs. The pages were removed from main on October 23, so both URLs return 404 and the sample stops at the first download.

Description & Review Guide

  • What are the major changes? The two URLs point to commit caf5bacd of MicrosoftDocs/semantic-kernel-docs, the last commit of main that has the pages.
  • What is the impact of these changes? The sample downloads the same pages as before. A fixed commit also keeps working if the pages move again.
  • What do you want reviewers to focus on? The pinned commit: it keeps the content that the questions of the sample were written for.

Tested:

  • Before: the sample stops at the first download with HttpRequestException: Response status code does not indicate success: 404 (Not Found).
  • After: both pinned URLs return 200, and the sample gets past both downloads.

Related Issue

Fixes #9647 (trivial change: two URLs in a sample)

AI Assistance

  • No material AI assistance was used.
  • This is an AI-assisted contribution. I reviewed, understood, and verified all submitted content and accept responsibility for it.

AI assistance details: the change, its verification and this PR description were prepared with an AI coding agent.

Contribution Checklist

  • The code builds clean without any errors or warnings
  • All unit tests pass, and I have added new tests where possible
  • The PR follows the Contribution Guidelines
  • This PR links to an agreed issue with no competing open PR, or the Related Issue section documents a trivial-change or repository-automation exception.
  • This is not a breaking change. If it is a breaking change, add the breaking change label (or add "[BREAKING]" to the title prefix, before or after any language prefix) — a workflow keeps the label and title prefix in sync automatically.

This branch was successfully deployed

1 active deployment
github-app-auth — 49e1e2b5 Deployed Oct 23, 2026 by Laurianti via team_check #7334
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

.NET Usage: [Issues, PRs], Target: .Net

Projects

None yet

Development

Successfully merging this pull request may close these issues.

.NET: [Bug]: RAG Step02 sample fails with 404 on the documentation pages

1 participant