Skip to content

Rename ChatMessageContentPart's factory methods#225

Merged
joseharriaga merged 10 commits into
mainfrom
joseharriaga/ChatMessageContentPart
Sep 4, 2024
Merged

Rename ChatMessageContentPart's factory methods#225
joseharriaga merged 10 commits into
mainfrom
joseharriaga/ChatMessageContentPart

Conversation

@joseharriaga
Copy link
Copy Markdown
Owner

@joseharriaga joseharriaga commented Aug 27, 2024

  • Renamed ChatMessageContentPart's CreateTextMessageContentPart factory method to CreateTextPart.
  • Renamed ChatMessageContentPart's CreateImageMessageContentPart factory method to CreateImagePart.
  • Renamed ChatMessageContentPart's CreateRefusalMessageContentPart factory method to CreateRefusalPart.
  • Renamed ImageChatMessageContentPartDetail to ChatImageDetailLevel.
  • Removed ChatMessageContentPart's ToString overload. It does not receive a lot of usage, and it can also return null against the ToString guidelines.
  • Added better doc comments.

@joseharriaga joseharriaga force-pushed the joseharriaga/ChatMessageContentPart branch from 8d0ba25 to 06d1020 Compare August 27, 2024 23:50
Comment thread .dotnet/src/Custom/Chat/ChatMessageContentPart.Serialization.cs Outdated
Comment thread .dotnet/src/Custom/Chat/ChatMessageContentPart.cs Outdated
Comment thread .dotnet/src/Custom/Chat/ChatMessageContentPart.cs Outdated
Comment thread .dotnet/examples/Chat/Example01_SimpleChat.cs Outdated
@joseharriaga joseharriaga force-pushed the joseharriaga/ChatMessageContentPart branch from 06d1020 to 25c1daa Compare September 4, 2024 23:01
@joseharriaga joseharriaga merged commit 49e3f42 into main Sep 4, 2024
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.

3 participants