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.

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

Update the conversation status as closed #216

Open
Oceania2018 opened this issue Nov 22, 2023 · 2 comments
Open

Update the conversation status as closed #216

Oceania2018 opened this issue Nov 22, 2023 · 2 comments

Comments

@Oceania2018
Copy link
Member

A Conversation Hook needs to be implemented to update Conversation to the closed state.

The conversation will be closed in two situations:

  • When the conversation ends normally and is triggered by LLM,;
  • When the idle time is exceeded and the conversation is forcibly ended by the background;

No matter which situation is triggered, the OnConversationEnding method in IConversationHook will be called.

Therefore we can implement a new conversation hook here and update the conversation status to ConversationStatus.Closed in the OnConversationEnding function.

@wilson-dixon
Copy link

@Oceania2018 was this implemented?

@Oceania2018
Copy link
Member Author

@waelhosn I think it's been implemented from this PR. Let me know if it's working or not.

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

No branches or pull requests

2 participants