Skip to content

.NET: Rename 3rdPartyThreadStorage sample to 3rdPartyChatHistoryStorage - #3643

Merged
westey (westey-m) merged 4 commits into
microsoft:mainfrom
westey-m:rename-3rdpartythreadstorage
Feb 4, 2026
Merged

.NET: Rename 3rdPartyThreadStorage sample to 3rdPartyChatHistoryStorage#3643
westey (westey-m) merged 4 commits into
microsoft:mainfrom
westey-m:rename-3rdpartythreadstorage

Conversation

@westey-m

Copy link
Copy Markdown
Contributor

Motivation and Context

#3431

Description

  • Rename 3rdPartyThreadStorage sample to 3rdPartyChatHistoryStorage

Contribution Checklist

  • The code builds clean without any errors or warnings
  • The PR follows the Contribution Guidelines
  • All unit tests pass, and I have added new tests where possible
  • Is this a breaking change? If yes, add "[BREAKING]" prefix to the title of the PR.

CopilotAI review requested due to automatic review settings February 3, 2026 13:09
@markwallace-microsoftMark Wallace (markwallace-microsoft) added documentation Usage: [Issues, PRs], Target: documentation in the code base and learn docs .NET Usage: [Issues, PRs], Target: .Net labels Feb 3, 2026
@github-actionsgithub-actionsBot changed the title Rename 3rdPartyThreadStorage sample to 3rdPartyChatHistoryStorage.NET: Rename 3rdPartyThreadStorage sample to 3rdPartyChatHistoryStorageFeb 3, 2026

CopilotAI 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 renames the Agent_Step07_3rdPartyThreadStorage sample to Agent_Step07_3rdPartyChatHistoryStorage to align with the terminology change from "thread storage" to "chat history storage" as requested in issue #3431.

Changes:

  • Renamed the sample directory from Agent_Step07_3rdPartyThreadStorage to Agent_Step07_3rdPartyChatHistoryStorage
  • Updated all references to the sample in documentation and solution files
  • Enhanced code comments to better explain the sample's purpose and functionality

Reviewed changes

Copilot reviewed 3 out of 4 changed files in this pull request and generated no comments.

FileDescription
dotnet/samples/GettingStarted/Agents/README.mdUpdated table entry to reference the renamed sample with new terminology
dotnet/samples/GettingStarted/Agents/Agent_Step07_3rdPartyChatHistoryStorage/Program.csEnhanced header comments to better explain the custom ChatHistoryProvider and session state persistence
dotnet/samples/GettingStarted/Agents/Agent_Step07_3rdPartyChatHistoryStorage/Agent_Step07_3rdPartyChatHistoryStorage.csprojProject file for the renamed sample (consistent with other samples)
dotnet/agent-framework-dotnet.slnxUpdated solution reference to point to the renamed project

@westey-m
westey (westey-m) added this pull request to the merge queueFeb 4, 2026
@github-merge-queue
github-merge-queueBot removed this pull request from the merge queue due to a conflict with the base branch Feb 4, 2026
@westey-m
westey (westey-m) added this pull request to the merge queueFeb 4, 2026
@github-merge-queue
github-merge-queueBot removed this pull request from the merge queue due to failed status checks Feb 4, 2026
@westey-m
westey (westey-m) added this pull request to the merge queueFeb 4, 2026
Merged via the queue into microsoft:main with commit 5e565dbFeb 4, 2026
15 checks passed
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentationUsage: [Issues, PRs], Target: documentation in the code base and learn docs.NETUsage: [Issues, PRs], Target: .Net

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants

@westey-m@rogerbarreto@SergeyMenshykh@markwallace-microsoft