Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[.Net][Feature Request]: Deprecate IStreamingMessage #3045

Closed
LittleLittleCloud opened this issue Jun 30, 2024 · 0 comments · Fixed by #3047
Closed

[.Net][Feature Request]: Deprecate IStreamingMessage #3045

LittleLittleCloud opened this issue Jun 30, 2024 · 0 comments · Fixed by #3047
Assignees
@LittleLittleCloud
Labels
api-break-change Will break existing api and needs an migration for previous code to work AutoGen.Net issues related to AutoGen.Net enhancement New feature or request
Milestone
AutoGen.Net 0.0.16

Comments

@LittleLittleCloud
Copy link
Collaborator

LittleLittleCloud commented Jun 30, 2024 •
edited
Loading

Is your feature request related to a problem? Please describe.

The functionality of IStreamingMessage can be perfectly replaced by IMessage

Migration guide

Deprecate IStreamingMessage will introduce breaking change to existing code that consumes AutoGen.Net, especially for the following types

  • IStreamingAgent
  • IStreamingMiddleware

To migrate towards latest AutoGen.Net, you can replace all IStreamingMessage and IStreamingMessage<T> with IMessage and IMessage<T>

Also, you can refer to #3047 on how the tests being migrates for more reference

Additional context

No response
@LittleLittleCloud LittleLittleCloud added the enhancement New feature or request label Jun 30, 2024
@LittleLittleCloud LittleLittleCloud self-assigned this Jun 30, 2024
@LittleLittleCloud LittleLittleCloud added this to the AutoGen.Net 0.0.16 milestone Jun 30, 2024
@LittleLittleCloud LittleLittleCloud mentioned this issue Jun 30, 2024
Merged
3 tasks
@LittleLittleCloud LittleLittleCloud added api-break-change Will break existing api and needs an migration for previous code to work AutoGen.Net issues related to AutoGen.Net labels Jun 30, 2024
github-merge-queue bot pushed a commit that referenced this issue Jul 1, 2024
@LittleLittleCloud
[.Net] Fix #3045 (#3047)
66e5061 
* make IStreamingMessage obsolete

* update final reply message
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@LittleLittleCloud LittleLittleCloud

Labels
api-break-change Will break existing api and needs an migration for previous code to work AutoGen.Net issues related to AutoGen.Net enhancement New feature or request
Projects
None yet
Milestone
AutoGen.Net 0.0.16
Development

Successfully merging a pull request may close this issue.

[.Net] Fix #3045 microsoft/autogen
1 participant
@LittleLittleCloud